dylang / grunt-notify

Automatic Notifications when Grunt tasks fail.
https://npmjs.org/package/grunt-notify
MIT License
921 stars 73 forks source link

Fixed several problems running Growl for Windows #94

Open johnmcase opened 9 years ago

johnmcase commented 9 years ago

Fixes several issues on Windows. Based on Pull Request #73 with some additional updates by me.

Fixes #91 and #36

All tests pass for me on Windows 7 SP1 running node 0.12.0

matttrawicki commented 9 years ago

:+1: Haven't gotten a growl since updating to node 0.12.0. Thanks @johnmcase for submitting this PR.

dylang commented 9 years ago

Thanks @johnmcase! @matttrawicki, have you tried the PR to see if it fixes the problem for you?

matttrawicki commented 9 years ago

Yes, notifications are growling with these changes.

lamchau commented 9 years ago

I found this issue while updating to node 0.12 as well, changes also work for me. Thank you for the fix!