Failed on bootstrap/js/*.js about gulp-jscs HOT 9 CLOSED

Working as intended. It's supposed to warn you on code formatting violations, which is what it's doing.

from gulp-jscs.

I cann't understand this.
Why style checker crashe task. If i run this task from watch... Crash task, watch, gulp.
As for me it's no sense. Style checker: check style, write errors thats all.
I must be able to build project with js style errors or warnings.

from gulp-jscs.

Please read the Gulp documentation. You can handle the error if you want.

https://github.com/gulpjs/gulp/wiki#articles-and-recipes

from gulp-jscs.

I don't need handle error from checker. I want stream, pipe... Checker should not break stream.
JSHint, example not crached stream. Check and write error. That's all.

gulp.task('build:scripts', function() {
  gulp.src(path.join(app.scripts.src, '**/*.js'))
    .pipe(jshint())
    .pipe(jshint.reporter())
    .pipe(jscs())
    .pipe(gulp.env.production ? tasks.uglify() : tasks.util.noop())
    .pipe(gulp.dest(app.scripts.dst));
});

I read https://github.com/gulpjs/gulp/wiki#articles-and-recipes.
First trick write after dissqussion my issue gulpjs/gulp#82
Second trick has no relation to handle errors.

from gulp-jscs.

@contra

from gulp-jscs.

@6aKa gulp-jshint is planned to start emitting errors as well. You need to handle the error yourself like this

gulp.task('build:scripts', function() {
  gulp.src(path.join(app.scripts.src, '**/*.js'))
    .pipe(jshint())
    .pipe(jshint.reporter())
    .pipe(jscs().on('error', gutil.log))
    .pipe(gulp.env.production ? tasks.uglify() : tasks.util.noop())
    .pipe(gulp.dest(app.scripts.dst));
});

Keep in mind that gulp is standard node and doesn't have any magic error handling. Plugins will emit errors, you need to handle them.

from gulp-jscs.

@sindresorhus You should use the new PluginError stuff from gulp-util to lose the stack on these

from gulp-jscs.

@contra Yeah, I'm planning to go through my plugins, but waiting on the streaming changes to be sorted out first.

from gulp-jscs.

@sindresorhus We're leaving everything where it is. Basically all that has changed is that if you are a fundamentally buffered library and you want to support streams you can use @nfroidure's new BufferStream module to make it easy

Took quite a clusterfk to get to that decision but I'm confident in it

from gulp-jscs.