-
Notifications
You must be signed in to change notification settings - Fork 32
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
Plugin crashes gulp on syntax error #15
Comments
the gulpfile looks good, could you paste the content of |
It occurred during editing. I do not know what it was. But sometimes it |
Error is in my issue description above
So I think a plugin must handle it somehow. |
did you try this? gulp.task('jade', function (cb) {
return gulp.src(['**/*.jade'], {cwd: app + 'jade'})
.pipe($.plumber(options.plumber))
.pipe($.cached('jade'))
.pipe($.if(global.isWatching, jadeInheritance({basedir: app + 'jade'})))
.pipe($.filter(function (file) {
return !/\/_/.test(file.path) && !/^_/.test(file.relative);
}))
.on('error', gutil.log)) //handle error
.pipe($.jade(options.jade))
.pipe($.prettify(options.htmlPrettify))
.pipe($.rename(function (path) {
path.extname = ".blade.php";
}))
.pipe(gulp.dest(app + 'views'))
.pipe($.size({title: 'jade'}))
.pipe(browserSync.reload({stream: true}));
}); |
Ok, I'll try |
This one do not run even if remove redundant bracket. |
sorry for the delay in my answers I think that I've fixed the problem, could you upgrade to gulp-jade-inheritance 0.5.5? thanks |
I cannot upgrade because 0.5.5 right now is unavailable |
It should be available now |
The plugin feeds an invalid glob pattern to the output chain if some errors occur with jade syntax.
(to next gulp.src())
My task:
I think it must not kill a gulp process.
The text was updated successfully, but these errors were encountered: