Skip to content

JSCS reporter compatibility #113

Open
@VWoeltjen

Description

@VWoeltjen

I'm having some compatibility issues when trying to use the jscs-html-reporter

This works fine:

$ jscs src/ --reporter jscs-html-reporter
>> jscs report written to /.../jscs-html-report.html
Too many errors... Increase `maxErrors` configuration option value to see more.

But this does not:

gulp.task('checkstyle', function () {
    return gulp.src(paths.scripts)
        .pipe(jscs())
        .pipe(jscs.reporter('jscs-html-reporter'));
});

When I run this task, I hit an error with that reporter:

$ gulp checkstyle
[13:03:42] Using gulpfile ~/.../gulpfile.js
[13:03:42] Starting 'checkstyle'...
/.../jscs-html-reporter.js:14
    var config = this.options || this,
                     ^

TypeError: Cannot read property 'options' of undefined
  at module.exports (/.../node_modules/jscs-html-reporter/jscs-html-reporter.js:14:22)
  at DestroyableTransform._transform (/.../node_modules/gulp-jscs/reporters/index.js:21:4)

(Full stack trace omitted for brevity.)

I'm guessing there is some difference between the way jscs normally invokes a reporter, and the way it's getting invoked from gulp-jscs. It would be useful to have sufficient compatibility with vanilla jscs to use this reporter.

I have found the gulp-jscs-with-reporter and gulp-jscs-html-reporter projects, which provide a workaround to this, but I would prefer not to switch to a fork of gulp-jscs for this one feature.

Metadata

Metadata

Assignees

No one assigned

    Labels

    No labels
    No labels

    Type

    No type

    Projects

    No projects

    Milestone

    No milestone

    Relationships

    None yet

    Development

    No branches or pull requests

    Issue actions