-
Notifications
You must be signed in to change notification settings - Fork 12
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
Update terminal notification setup #337
Comments
I suppose one could try to conditionally disable the plugin on Apple Silicon - possibly using Node.js' |
…under the assumption it's a runtime error, not one at install-time. I don't have an M1, so I can't tell. 🙂 |
👋 to be honest, I have no idea what the error is about or where it's triggered. But I have an M1 mac at hand and could test whatever you want me to test :) |
Thanks @pmeinhardt & @tessi 💚 I also have an M1 and could test it there. But I prefer to have the setup run quick and friction-less for anyone looking to contribute. The extra complexity and respective maintenance effort just for the sake of the build notification does not feel worth the effort to me. The time would be better spend speeding up the build itself 😉 Therefore going with simply removing the notifier 🧹 |
As it turned out in #336, the
webpack-build-notifier
package we're using in development isn't compatible with Apple's new M1 architecture and probably won't in the near future. See this issue for some background on this.How should we proceed on this:
The text was updated successfully, but these errors were encountered: