tools: set eslint comma-spacing to 'warn'

Certain cases with comments inside arrays or object literals fail to
pass eslint's comma-spacing rule. This change sets the comma-spacing
rule to the 'warn' level.

Once https://github.com/eslint/eslint/issues/2408 is resolved and
released, this rule should be set back to 'error' level.

PR-URL: https://github.com/iojs/io.js/pull/1672
Reviewed-By: Yosuke Furukawa <yosuke.furukawa@gmail.com>

diff --git a/.eslintrc b/.eslintrc
index 8d93ac2..8c7ffdc 100644 (file)
--- a/.eslintrc
+++ b/.eslintrc
@@ -46,7 +46,8 @@ rules:
     - 2
     - 2
   ## add space after comma
-  comma-spacing: 2
+  ## set to 'warn' because of https://github.com/eslint/eslint/issues/2408
+  comma-spacing: 1
   ## put semi-colon
   semi: 2
   ## require spaces operator like var sum = 1 + 1;
@@ -56,21 +57,20 @@ rules:
   ## require parens for Constructor
   new-parens: 2
   ## max 80 length
-  max-len: 
+  max-len:
     - 2
     - 80
     - 2
 
-
   # Strict Mode
   # list: https://github.com/eslint/eslint/tree/master/docs/rules#strict-mode
   ## 'use strict' on top
-  strict: 
+  strict:
     - 2
     - "global"
 
 # Global scoped method and vars
-globals: 
+globals:
   DTRACE_HTTP_CLIENT_REQUEST: true
   LTTNG_HTTP_CLIENT_REQUEST: true
   COUNTER_HTTP_CLIENT_REQUEST: true
@@ -89,4 +89,3 @@ globals:
   DTRACE_NET_SERVER_CONNECTION: true
   LTTNG_NET_SERVER_CONNECTION: true
   COUNTER_NET_SERVER_CONNECTION: true
-