tools: enable eslint strict key-spacing

BridgeAR · MayaLekova · commit 5e1d9748cec3 · 2018-05-08T17:35:24.000+02:00
PR-URL: nodejs#18831 Reviewed-By: Luigi Pinca <luigipinca@gmail.com> Reviewed-By: Matheus Marchini <matheus@sthima.com> Reviewed-By: Anna Henningsen <anna@addaleax.net> Reviewed-By: Michaël Zasso <targos@protonmail.com> Reviewed-By: Anatoli Papirovski <apapirovski@mac.com> Reviewed-By: Colin Ihrig <cjihrig@gmail.com>
diff --git a/.eslintrc.js b/.eslintrc.js
@@ -166,7 +166,7 @@ module.exports = {
       ObjectExpression: 'first',
       SwitchCase: 1,
     }],
-    'key-spacing': ['error', { mode: 'minimum' }],
+    'key-spacing': ['error', { mode: 'strict' }],
     'keyword-spacing': 'error',
     'linebreak-style': ['error', 'unix'],
     'max-len': ['error', {
diff --git a/benchmark/http/_chunky_http_client.js b/benchmark/http/_chunky_http_client.js
@@ -5,8 +5,8 @@ const common = require('../common.js');
 const net = require('net');
 
 const bench = common.createBenchmark(main, {
-  len:  [1, 4, 8, 16, 32, 64, 128],
-  n:  [5, 50, 500, 2000],
+  len: [1, 4, 8, 16, 32, 64, 128],
+  n: [5, 50, 500, 2000],
   type: ['send'],
 });
 
diff --git a/test/parallel/test-eslint-require-buffer.js b/test/parallel/test-eslint-require-buffer.js
@@ -36,14 +36,14 @@ ruleTester.run('require-buffer', rule, {
       output: useStrict + bufferModule + useBuffer,
     },
     {
-      code:  mockComment + useBuffer,
+      code: mockComment + useBuffer,
       errors: [{ message }],
-      output:  mockComment + bufferModule + useBuffer,
+      output: mockComment + bufferModule + useBuffer,
     },
     {
-      code:  mockComment + useStrict + useBuffer,
+      code: mockComment + useStrict + useBuffer,
       errors: [{ message }],
-      output:  mockComment + useStrict + bufferModule + useBuffer,
+      output: mockComment + useStrict + bufferModule + useBuffer,
     },
   ]
 });
diff --git a/test/parallel/test-https-agent-servername.js b/test/parallel/test-https-agent-servername.js
@@ -10,7 +10,7 @@ const fixtures = require('../common/fixtures');
 const options = {
   key: fixtures.readKey('agent1-key.pem'),
   cert: fixtures.readKey('agent1-cert.pem'),
-  ca:  fixtures.readKey('ca1-cert.pem')
+  ca: fixtures.readKey('ca1-cert.pem')
 };
 
 

Original file line number	Diff line number	Diff line change
`@@ -36,14 +36,14 @@ ruleTester.run('require-buffer', rule, {`
`36`	`36`	`output: useStrict + bufferModule + useBuffer,`
`37`	`37`	`},`
`38`	`38`	`{`
`39`		`- code: mockComment + useBuffer,`
	`39`	`+ code: mockComment + useBuffer,`
`40`	`40`	`errors: [{ message }],`
`41`		`- output: mockComment + bufferModule + useBuffer,`
	`41`	`+ output: mockComment + bufferModule + useBuffer,`
`42`	`42`	`},`
`43`	`43`	`{`
`44`		`- code: mockComment + useStrict + useBuffer,`
	`44`	`+ code: mockComment + useStrict + useBuffer,`
`45`	`45`	`errors: [{ message }],`
`46`		`- output: mockComment + useStrict + bufferModule + useBuffer,`
	`46`	`+ output: mockComment + useStrict + bufferModule + useBuffer,`
`47`	`47`	`},`
`48`	`48`	`]`
`49`	`49`	`});`