Turbo87 / webpack-notifier

webpack + node-notifier = build status system notifications
ISC License
306 stars 41 forks source link

Updates node-notifier dependency to v5 #23

Closed mikaelbr closed 7 years ago

mikaelbr commented 7 years ago

Updates the dependency of node-notifier to the latest newly released v5.0.1.

With this change there are a lot fewer dependencies as the built in CLI is removed, and other dependencies are trimmed. In addition it gives better Windows support with custom sounds and more stable activation/timeout.

For macOS it adds ability to reply/add buttons.

As you provide ability to pass through options all the way to node-notifier I'd recommend doing a minor or even major release here. It shouldn't have broken the API, but it returns different output and I don't want to guarantee that some existing use case works exactly the same way due to the nature of the changes.

Turbo87 commented 7 years ago

awesome, thanks @mikaelbr!