lnikkila / jshint-reporter-jscs

JSHint reporter with code preview like JSCS.

jshint-reporter-jscs

JSHint reporter that looks like the reporter from JSCS. Supports code previews with line numbering.

Installation

$ npm install jshint-reporter-jscs --save-dev

Usage with gulp

return gulp.src(scripts)
    .pipe(jshint())
    .pipe(jshint.reporter('jshint-reporter-jscs'));

jshint-reporter-jscs's Issues

malformed quotes breaks the reporter

with this code

var iFrameHTML = '<iframe src="'+ iFrameSrc + "></iframe>';

I know it's broken code but shouldn't the reporter notify me of that instead of breaking on my face?

/Users/lbueno/work/www/shared-auth/devkit/node_modules/jshint-reporter-jscs/source/reporter.js:110
  var dashes = new Array(column + 9).join('-');
               ^

RangeError: Invalid array length
    at writeArrow (/Users/lbueno/work/www/shared-auth/devkit/node_modules/jshint-reporter-jscs/source/reporter.js:110:16)
    at writeEntry (/Users/lbueno/work/www/shared-auth/devkit/node_modules/jshint-reporter-jscs/source/reporter.js:50:13)
    at Array.map (native)
    at reportErrors (/Users/lbueno/work/www/shared-auth/devkit/node_modules/jshint-reporter-jscs/source/reporter.js:18:11)
    at DestroyableTransform.<anonymous> (/Users/lbueno/work/www/shared-auth/devkit/node_modules/gulp-jshint/src/reporters/index.js:48:7)
    at DestroyableTransform._transform (/Users/lbueno/work/www/shared-auth/devkit/node_modules/gulp-jshint/src/stream.js:26:15)
    at DestroyableTransform.Transform._read (/Users/lbueno/work/www/shared-auth/devkit/node_modules/gulp-jshint/node_modules/readable-stream/lib/_stream_transform.js:184:10)
    at DestroyableTransform.Transform._write (/Users/lbueno/work/www/shared-auth/devkit/node_modules/gulp-jshint/node_modules/readable-stream/lib/_stream_transform.js:172:12)
    at doWrite (/Users/lbueno/work/www/shared-auth/devkit/node_modules/gulp-jshint/node_modules/readable-stream/lib/_stream_writable.js:237:10)
    at writeOrBuffer (/Users/lbueno/work/www/shared-auth/devkit/node_modules/gulp-jshint/node_modules/readable-stream/lib/_stream_writable.js:227:5)
    at DestroyableTransform.Writable.write (/Users/lbueno/work/www/shared-auth/devkit/node_modules/gulp-jshint/node_modules/readable-stream/lib/_stream_writable.js:194:11)
    at write (/Users/lbueno/work/www/shared-auth/devkit/node_modules/gulp-jshint/node_modules/readable-stream/lib/_stream_readable.js:623:24)
    at flow (/Users/lbueno/work/www/shared-auth/devkit/node_modules/gulp-jshint/node_modules/readable-stream/lib/_stream_readable.js:632:7)
    at DestroyableTransform.pipeOnReadable (/Users/lbueno/work/www/shared-auth/devkit/node_modules/gulp-jshint/node_modules/readable-stream/lib/_stream_readable.js:664:5)
    at emitNone (events.js:72:20)
    at DestroyableTransform.emit (events.js:166:7)

I've got gulp-plumber working properly and if I comment out the reporter it works fine, so... 😞

BTW awesome work! ;)

many thanks indeed!