From f07b3b600b21f38783d7f889619a572b00c71247 Mon Sep 17 00:00:00 2001 From: Roman Reiss Date: Mon, 11 May 2015 14:24:38 +0200 Subject: [PATCH] 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 --- .eslintrc | 11 +++++------ 1 file changed, 5 insertions(+), 6 deletions(-) diff --git a/.eslintrc b/.eslintrc index 8d93ac2738278e..8c7ffdc27b3d44 100644 --- 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 -