-
-
Notifications
You must be signed in to change notification settings - Fork 303
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
SyntaxError in plugin "gulp-notify" #148
Comments
Your configuration might have a syntax error. Did you check that? |
Hmmm I think it's all ok...
|
The error you're getting and the fact you made config changes makes me think this is a config issue, but your config looks fine and the config file isn't JSON (which the error points to). If you go back to the default config file, do you still have the issue? |
Changed back to the default config file and still getting the same issue! |
As it seems to be a issue with the notification system, is there a way I can just disable this? messages in the command line are enough for me! |
Feel free to contribute a setting that is checked when writing messages.
Your contributions are welcomed.
… |
Bug Report
I've updated the config file and when I run
npm start
everything seems to run ok the first time (and compiles SASS)... but then when it reaches the end (afterSTYLES - completed!
I get the error and the task doesn't seem to continue watching/compiling files:I'm on OSX Mojave 10.14.5 and using a Docker container.
I've installed
notify-send
in the project withnpm install notify-send
but this has made no difference. I've also installedlibnotify
globally on my system withbrew install libnotify
but this has also not fixed the issue.I've also tried adding double quotes around the properties as mentioned in mikaelbr/gulp-notify#129
node -v
: v10.16.0npm -v
: 6.9.0.The text was updated successfully, but these errors were encountered: