failAfterError options displays an error message but doesn't exit with an error code about gulp-stylelint HOT 3 CLOSED

Investigating.

from gulp-stylelint.

I've run a test case locally and failAfterError does cause the task to fail with non-zero code on error.

After looking at your code I noticed that you're using gulp-plumber. It captures the error emitted by gulp-stylelint, logs it to console and ends the stream. So the actual error is caught but not re-thrown, thus you don't get a non-zero error code.

I'm not an expert in best practices on writing pipes, but this is not a long-running task, so you could add a process.exit(1); just after this.emit('end'); in your error handler.

from gulp-stylelint.

@olegskl Yes you are absolutely right. Having removed gulp-plumber from the task, failAfterError exits with the correct error code. Sorry to waste your time, thanks for looking into it 👍

from gulp-stylelint.