Browse Source

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>
v2.0.2
Roman Reiss 10 years ago
parent
commit
f07b3b600b
  1. 11
      .eslintrc

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

Loading…
Cancel
Save