Reinstate `gulp-notify`

uswds / uswds-gulp

A simple Gulp 4.0 workflow for transforming USWDS Sass into browser-readable CSS.

Other

35 stars 17 forks source link

Reinstate `gulp-notify` #10

Closed thisisdano closed 2 years ago

thisisdano commented 5 years ago

As we saw in #8, using gulp-notify was causing a syntax error:

[12:11:35] gulp-notify: [Error in notifier] SyntaxError in plugin "gulp-notify"
Message:
    Unexpected token { in JSON at position 129

This appears to be an upstream error, captured in mikaelbr/gulp-notify#129 When this upstream error is resolved, we should reinstate gulp-notify so everyone can enjoy hearing that satisfying "pop" sound again.

mejiaj commented 4 years ago

The last release for gulp-notify was in Jan 2018.

Maybe we should consider node-notifier, which looks to be under active development.

thisisdano commented 2 years ago

Closed as stale