1berry/nodejs
1
0
Fork 0
Code Issues Packages Projects Releases Wiki Activity

tools: re-enable comma-spacing linter rule

Browse Source 
The rule was disabled because of an eslint bug which is now resolved.
All code in lib was already conforming and only test code needed a few
changes to make the linter happy with this rule enabled.

Ref: https://github.com/eslint/eslint/issues/2408

PR-URL: https://github.com/nodejs/io.js/pull/2072
Reviewed-By: Yosuke Furukawa <yosuke.furukawa@gmail.com>
Reviewed-by: Colin Ihrig <cjihrig@gmail.com>
Reviewed-By: Alex Kocharin <alex@kocharin.ru>
This commit is contained in:
Roman Reiss 2015-06-28 17:42:35 +02:00
parent d91e10b3bd
commit e3f9335c40
11 changed files with 17 additions and 18 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

3
.eslintrc
View File

@ -43,8 +43,7 @@ rules:
## 2 space indentation ## 2 space indentation
indent: [2, 2] indent: [2, 2]
## add space after comma ## add space after comma
## set to 'warn' because of https://github.com/eslint/eslint/issues/2408 comma-spacing: 2
comma-spacing: 1
## put semi-colon ## put semi-colon
semi: 2 semi: 2
## require spaces operator like var sum = 1 + 1; ## require spaces operator like var sum = 1 + 1;

2
test/parallel/test-fs-mkdir.js
View File

@ -36,7 +36,7 @@ common.refreshTmpDir();
unlink(pathname); unlink(pathname);
fs.mkdir(pathname, 511 /*=0777*/, function(err) { fs.mkdir(pathname, 0o777, function(err) {
assert.equal(err, null); assert.equal(err, null);
assert.equal(common.fileExists(pathname), true); assert.equal(common.fileExists(pathname), true);
ncalls++; ncalls++;