Fix rubocop Rake/MethodDefinitionInTask warnings

.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.

lib/tasks/eslint.rake

@@ -1,20 +1,20 @@
 task "eslint" => "eslint:check"
 
+def yarn_path
+  Rails.root.join("bin/yarn").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
+end
+
 namespace "eslint" do
-  def yarn_path
-    Rails.root.join("bin/yarn").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
-  end
-
   task :check => :environment do
     system(yarn_path, "run", "eslint", "-c", config_file, *js_files) || abort
   end