tools: update eslint config

The no-reserved-keys rule doesn't exist anymore and we don't need ES3
compatibility.
escape and unescape are now known by eslint.
--reset flag was removed and it is now the default behavior.

PR-URL: https://github.com/nodejs/io.js/pull/2286
Reviewed-By: Roman Reiss <me@silverwind.io>

diff --git a/.eslintrc b/.eslintrc
index 33a7919d88c9ef6cd27bf7ccdc8134388f8e6e40..2197bf35ccf7bed98bfbbcbac9075551ef786469 100644 (file)
--- a/.eslintrc
+++ b/.eslintrc
@@ -27,8 +27,6 @@ rules:
   no-duplicate-case: 2
   ## disallow assignment of exceptional params
   no-ex-assign: 2
-  ## disallow use of reserved words as keys like enum, class
-  no-reserved-keys: 2
   ## disallow unreachable code
   no-unreachable: 2
   ## require valid typeof compared string like typeof foo === 'strnig'
@@ -44,7 +42,7 @@ rules:
   ## use single quote, we can use double quote when escape chars
   quotes: [2, "single", "avoid-escape"]
   ## 2 space indentation
-  indent: [2, 2]
+  indent: [2, 2, {SwitchCase: 1}]
   ## add space after comma
   comma-spacing: 2
   ## put semi-colon
@@ -112,5 +110,3 @@ globals:
   DTRACE_NET_SERVER_CONNECTION         : false
   LTTNG_NET_SERVER_CONNECTION          : false
   COUNTER_NET_SERVER_CONNECTION        : false
-  escape                               : false
-  unescape                             : false

diff --git a/Makefile b/Makefile
index 94eb41995367eda2b61fe2558cad41de195306fa..02f4363ab35832d342849b65054c77994ab28ed7 100644 (file)
--- a/Makefile
+++ b/Makefile
@@ -510,7 +510,7 @@ bench-idle:
 
 jslint:
        $(NODE) tools/eslint/bin/eslint.js src lib test tools/eslint-rules \
-               --rulesdir tools/eslint-rules --reset --quiet
+               --rulesdir tools/eslint-rules --quiet
 
 CPPLINT_EXCLUDE ?=
 CPPLINT_EXCLUDE += src/node_lttng.cc

diff --git a/vcbuild.bat b/vcbuild.bat
index 4311dfdee01b237e23c4e90f0635c291ec28bf29..31fc8cd65964392acf3259e6f85ebd6803491c28 100644 (file)
--- a/vcbuild.bat
+++ b/vcbuild.bat
@@ -251,7 +251,7 @@ goto jslint
 :jslint
 if not defined jslint goto exit
 echo running jslint
-%config%\node tools\eslint\bin\eslint.js src lib test tools\eslint-rules --rulesdir tools\eslint-rules --reset --quiet
+%config%\node tools\eslint\bin\eslint.js src lib test tools\eslint-rules --rulesdir tools\eslint-rules --quiet
 goto exit
 
 :create-msvs-files-failed