browserify / watchify

watch mode for browserify builds
Other
1.79k stars 181 forks source link

Feature Request: After a failure, show a message on success #291

Closed ArtskydJ closed 8 years ago

ArtskydJ commented 8 years ago

When I use watchify, I'll sometimes get a parse error:

> watchify js/index.js -d -o public/bundle.js

Error: Parsing file C:\Users\Michael\Github\slideshow\js\index.js: Unexpected token (34:2)

I don't always see the error right away, and when I see it, I'm not sure if it was the most recent compile, or if it was fixed since... I would like to have it show a message like Success! when the issue is fixed.

Any thoughts on this? Would a PR be accepted?

ghost commented 8 years ago

You can use the -v switch to print a message every time the build completed successfully.

ArtskydJ commented 8 years ago

Because I tend to reflexively save often, a message every time is a bit much.

I'll probably start enabling the -v switch for my npm watch scripts.

Perhaps this requested feature is opinionated/out of scope. I'll close this issue. If a maintainer feels that it is in scope, feel free to re-open this issue, and I might make a PR.

mattdesl commented 8 years ago

You might like these:

eg:

es6