-
Notifications
You must be signed in to change notification settings - Fork 2.8k
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
Use npm script for brunch watch tasks #1273
Use npm script for brunch watch tasks #1273
Conversation
👍 |
Use npm script for brunch watch tasks
Thanks! |
This change actually breaks brunch usage for someone on windows :( I was troubleshooting a different brunch issue and had them try out the new npm script and watcher, and they get an eaacces error. We may need to roll this one back |
EACCESS sounds like a permission problem, is it possible it's unrelated? |
Yes, but it works fine under the previous watcher configuration, so it's not unrelated. The previous watcher config that this replaced was added specifically to workaround some perm issues we had on windows |
Ah, I'm sorry. I confirmed it working, but it was only on Windows 8, maybe there is some difference on different versions. |
yeah I believe you :) I'm not sure how prevalent the perm issue with be for windows with this change. Maybe an isolated case, but who knows |
Don't forget to also revert #1270 then! |
hypothesis: sounds like the kind of error you'd run into when the binstubs (node_modules/.bin) aren't made properly. maybe |
This is a follow-up to conversation that happened in #1270 . Moves the
brunch watch --stdin
call into an npm script.