-
-
Notifications
You must be signed in to change notification settings - Fork 5.6k
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
babel-cli postinstall spam #10152
Comments
Hey @tildechris! We really appreciate you taking the time to report an issue. The collaborators If you need any help, or just have general Babel or JavaScript questions, we have a vibrant Slack |
Good luck -) |
Thanks @zloirock. Hopefully this request remains civil. Your package obviously delivers value, but IMHO this is a regression. Please let me know if there's anything that would help you reconsider your decision. |
@tildechris Don't even worry about it - the people who want to push ads in inappropriate places always get spanked and this will be no different. People clearly don't like ads in general, never mind this desperate, pathetic and distasteful personal ad inappropriately spewed into your logs. Just look at the amount of people running adblock who disrupted Google and the rest of the big-money ad industry. You might consider even encouraging this bad behavior so we can reach endgame more quickly. If that's not enough for you right now, I would also encourage you and anyone else who cares to fill |
Thanks @waynebloss. In order to evaluate this issue effectively, we should keep commentary on the upsteam issue at zloirock/core-js#548. Let's limit this thread to its effect on babel-cli and possible solutions. |
@tildechris Yeah no problem. You won't see this particular account commenting there because apparently @zloirock can dish it out, but can't take it when people post inappropriate messages in inappropriate places that take up his or her resources. The possible solutions that I can see:
Eventually enough things will break and we'll win. There's no doubt in my mind. |
core-js is a major dependency, as it's used for a few plugins. Frankly, there's not a chance it's going to be removed. |
I can't reproduce this report: when I run I only get it from |
Thanks @nicolo-ribaudo. I should've added the only reason I thought it was babel-cli was because of
I agree that it's a reasonable compromise to only show this message if you directly depend on core-js. |
Fixed by zloirock/core-js#590 Current status:
|
Bug Report
Current Behavior
When installing babel-cli, a transitive dependency is spamming the installation log. Consider removing
core-js
as a dependency.Expected behavior/code
Installation without error should not produce any output by default.
Environment
cli
]Possible Solution
Consider removing
core-js
as a dependency.Additional context/Screenshots
Please see below for more context:
zloirock/core-js#548
tl;dr - The core-js maintainer isn't willing to revert this regression.
The text was updated successfully, but these errors were encountered: