Improve streams for Gulp 4 about gulp-stylelint HOT 4 OPEN

I think I might run in the same issue here. I'm trying to make the reporter fail on warnings, in order to make the CI build fail. But unfortunately my custom reporter isn't able to make the pipe fail.

from gulp-stylelint.

@brendanfalkowski The plugin emits a PluginError which causes the stream to fail with non-zero code on errors by default. You can test it on https://github.com/olegskl/gulp-stylelint-test. I don't know how splitting the linting and reporting into separate streams may resolve the issue, but I'm willing to discuss it if does not involve the condescending tone I noticed in the linked issue.

@nirazul Perhaps you can use a different configuration for your CI that elevates severity of warnings?

from gulp-stylelint.

@olegskl — I'm not sure exactly how splitting the streams helps, but I'm just passing on the feedback from @phated to find a solution and document getting a nicer local dev experience for others to reference. Consider me just a loving user of these tools, but I don't have the same depth with JS streams as the creators.

Something is handled differently because in my Gulp 3 build gulp-stylelint would not reach the success notification if linter errors were encountered. Example events:

1. Linter starts checking CSS
2. Errors were found
3. Errors are logged to the console
4. Notification fires (to MacOS) of the error

In Gulp 4, the errors are reported but a success notifier also fires (right after the failing notifier).

1. Linter starts checking CSS
2. Errors were found
3. Errors are logged to the console
4. Notification fires (to MacOS) of the error
5. Notification fires (to MacOS) of the success

I'm guessing this happens because the "lintCss" task is finishing, and that's what it's supposed to do when the linter runs and passes:

1. Linter starts checking CSS
2. All good
3. Notification fires (to MacOS) of the success

This could very well be my Gulpfile's patterns or my error formatter. Both of them changed between Gulp 3 and 4, but my new build is using fewer packages and more native Gulp/Node functions.

I was hoping it was a simple config change, but the options I tried didn't help:
gravitydepartment/frontend-starter#4 (comment)

from gulp-stylelint.

@olegskl I would prefer to keep one single configuration as I do with gulp-eslint. I think the only thing that needs to change is to take into consideration the --max-warnings flag of stylelint.

from gulp-stylelint.