Fix rubocop Rake/MethodDefinitionInTask warnings

diff --git a/.rubocop_todo.yml b/.rubocop_todo.yml
index f49440e81159d59165f8366b8a8d03a37ef4056f..3059d5773e55f609787cd5921735e35b7fa06732 100644 (file)
--- a/.rubocop_todo.yml
+++ b/.rubocop_todo.yml
@@ -178,11 +178,6 @@ Rake/Desc:
     - 'lib/tasks/subscribe_old_changesets.rake'
     - 'lib/tasks/testing.rake'
 
-# Offense count: 3
-Rake/MethodDefinitionInTask:
-  Exclude:
-    - 'lib/tasks/eslint.rake'
-
 # Offense count: 558
 # Cop supports --auto-correct.
 # Configuration parameters: EnforcedStyle.

diff --git a/lib/tasks/eslint.rake b/lib/tasks/eslint.rake
index 320651b5a3fd615818d56f600e5df359d2b69393..7f96531d9d0835ece408c9ad3dedc0d99fadfb1f 100644 (file)
--- a/lib/tasks/eslint.rake
+++ b/lib/tasks/eslint.rake
@@ -1,20 +1,20 @@
 task "eslint" => "eslint:check"
 
-namespace "eslint" do
-  def yarn_path
-    Rails.root.join("bin/yarn").to_s
-  end
+def yarn_path
+  Rails.root.join("bin/yarn").to_s
+end
 
-  def config_file
-    Rails.root.join("config/eslint.json").to_s
-  end
+def config_file
+  Rails.root.join("config/eslint.json").to_s
+end
 
-  def js_files
-    Rails.application.assets.each_file.select do |file|
-      file.ends_with?(".js") && !file.match?(%r{/(gems|vendor|i18n|node_modules)/})
-    end
+def js_files
+  Rails.application.assets.each_file.select do |file|
+    file.ends_with?(".js") && !file.match?(%r{/(gems|vendor|i18n|node_modules)/})
   end
+end
 
+namespace "eslint" do
   task :check => :environment do
     system(yarn_path, "run", "eslint", "-c", config_file, *js_files) || abort
   end