error on run about gulp-jscs

[Thomasdezeeuw]

I got the same error on version 0.4.0, goes wrong here: https://github.com/mdevils/node-jscs/blob/master/lib/checker.js#L27. Haven't looked into it any further.

[iamstarkov]

I thought it is my fault from #14 pull-request, in which I wanted to update jscs depency and I followed @deepak1556's advice here #14 (comment) and I think that all (except tests) went bad after this.

[Thomasdezeeuw]

I think in file: https://github.com/sindresorhus/gulp-jscs/blob/master/index.js#L13

checker.configure(config ? require(config) : configFile.load('.jscsrc', process.cwd()));
Should be:
checker.configure(configFile.load(config || '.jscsrc', process.cwd()));

[deepak1556]

What @Thomasdezeeuw suggested shld actually be enough as cli-config.js does the path resolving based on the directory provided or defaults to process.cwd() https://github.com/mdevils/node-jscs/blob/master/lib/cli-config.js#L19. So even better checker.configure(configFile.load(config || '.jscsrc'))

[iamstarkov]

I tried checker.configure(configFile.load(config || '.jscsrc', process.cwd())); and checker.configure(configFile.load(config || '.jscsrc')) and no one works for me with the same error:

/Users/vstarkov/projects/box/node_modules/gulp/node_modules/orchestrator/index.js:153
                        throw err;

[dstroot]

Same problem here...

[joeybaker]

FYI: this is caused by an breaking change in the update version of jscs. The default file name used to be .jscs.json. It's now .jscsrc. Change your file name, and you'll be all set.

[iamstarkov]

The default file name used to be .jscs.json. It's now .jscsrc. Change your file name, and you'll be all set.

I changed it in my #14 pull-request and shit happens

[iamstarkov]

thank you

[iamstarkov]

If anybody still get this error like this:

vstarkov@vstarkov-osx nodeChan(master)$ gulp jscs
[gulp] Using gulpfile /Users/vstarkov/projects/nodeChan/gulpfile.js
[gulp] Starting 'jscs'...
[gulp] 'jscs' errored after 45 ms Unexpected token }

/Users/vstarkov/projects/nodeChan/node_modules/gulp/node_modules/orchestrator/index.js:153
                        throw err;
                              ^
SyntaxError: Unexpected token }
    at Object.parse (native)
    at Object.exports.getContent (/Users/vstarkov/projects/nodeChan/node_modules/gulp-jscs/node_modules/jscs/lib/cli-config.js:55:28)
    at Object.exports.load (/Users/vstarkov/projects/nodeChan/node_modules/gulp-jscs/node_modules/jscs/lib/cli-config.js:83:20)
    at module.exports (/Users/vstarkov/projects/nodeChan/node_modules/gulp-jscs/index.js:13:35)
    at Gulp.getType (/Users/vstarkov/projects/nodeChan/gulpfile.js:28:11)
    at module.exports (/Users/vstarkov/projects/nodeChan/node_modules/gulp/node_modules/orchestrator/lib/runTask.js:31:7)
    at Gulp.Orchestrator._runTask (/Users/vstarkov/projects/nodeChan/node_modules/gulp/node_modules/orchestrator/index.js:273:3)
    at Gulp.Orchestrator._runStep (/Users/vstarkov/projects/nodeChan/node_modules/gulp/node_modules/orchestrator/index.js:214:10)
    at Gulp.Orchestrator.start (/Users/vstarkov/projects/nodeChan/node_modules/gulp/node_modules/orchestrator/index.js:134:8)
    at /usr/local/lib/node_modules/gulp/bin/gulp.js:77:20

it is because of 77 line in gulp.js, you can fix this error by installing latest gulp globally and locally.

@sindresorhus maybe this should be noted in README?