eslint

.gitignore

@@ -12,6 +12,7 @@ config/settings/*.local.yml
 coverage
 doc
 log
+node_modules
 public/assets
 public/attachments
 public/export

.travis.yml

@@ -26,8 +26,9 @@ before_script:
   - touch config/settings.local.yml
   - bundle exec rake db:migrate
   - bundle exec rake i18n:js:export
+  - bundle exec rake yarn:install
 script:
   - bundle exec rubocop -f fuubar
-  - bundle exec rake eslint:run_all
+  - bundle exec rake eslint
   - bundle exec erblint .
   - bundle exec rake test:db

config/eslint.json

@@ -33,6 +33,13 @@
     "eol-last": "error",
     "eqeqeq": ["error", "smart"],
     "func-call-spacing": "error",
+    "indent": ["error", 2, {
+      "SwitchCase": 1,
+      "VariableDeclarator": "first",
+      "FunctionDeclaration": { "parameters": "first" },
+      "FunctionExpression": { "parameters": "first" },
+      "CallExpression": { "arguments": "first" }
+    }],
     "key-spacing": "error",
     "keyword-spacing": "error",
     "no-alert": "warn",

lib/tasks/eslint.rake

@@ -0,0 +1,19 @@
+task "eslint" => "eslint:check"
+
+namespace "eslint" do
+  def js_files
+    require File.dirname(__FILE__) + "/../../config/environment"
+
+    Rails.application.assets.each_file.select do |file|
+      file.ends_with?(".js") && !file.match?(%r{/(gems|vendor|i18n)/})
+    end
+  end
+
+  task :check do
+    system(Rails.root.join("bin", "yarn").to_s, "exec", "eslint", "-c", "config/eslint.json", *js_files)
+  end
+
+  task :fix do
+    system(Rails.root.join("bin", "yarn").to_s, "exec", "eslint", "-c", "config/eslint.json", "--fix", *js_files)
+  end
+end

package.json

@@ -0,0 +1,9 @@
+{
+  "name": "openstreetmap",
+  "private": true,
+  "dependencies": {
+  },
+  "devDependencies": {
+    "eslint": "*"
+  }
+}