-
Notifications
You must be signed in to change notification settings - Fork 51
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
JSCS reporter compatibility #113
Comments
Supporting a jshint-like syntax for reporter options would also be useful, e.g.: gulp.task('checkstyle', function () {
return gulp.src(paths.scripts)
.pipe(jscs())
.pipe(jscs.reporter('jscs-html-reporter', {
reporterOutput: __dirname + "/jscs-report.html"
}));
}); |
@VWoeltjen just an FYI - I have added support for |
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
I'm having some compatibility issues when trying to use the jscs-html-reporter
This works fine:
But this does not:
When I run this task, I hit an error with that reporter:
(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.
The text was updated successfully, but these errors were encountered: