-
-
Notifications
You must be signed in to change notification settings - Fork 1.7k
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
This is a bit much #611
Comments
One request is fine but given the number of transitive dependencies in JS projects, one request per transitive require is exceedingly noisy (adds about 200 lines of output). I respect your right to add a call to action to the library you generously offer for free, but if other authors also choose to do this we can expect thousands of lines of output on every install. Thanks for core-js! I didn't even know I was using it :) |
See #597 |
Thanks! I saw this comment from another thread:
And that seems reasonable. Since you're keeping it in to thumb your nose at complainers, I would suggest going even bigger! Why not use https://www.npmjs.com/package/cowsay or make the output a rainbow of colors? Go big or go home, I say! 😄 |
Seems like desperation to request this often in an install. |
I have a solution. @davegear @Sequoia @pronebird modify npm postinstall.js file to block core-js module postinstall process. more detail: https://github.com/xqin/block-core-js-postinstall |
This is output 25 times:
The text was updated successfully, but these errors were encountered: