-
-
Notifications
You must be signed in to change notification settings - Fork 27k
New issue
Have a question about this project? # for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “#”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? # to your account
Remove bundledDependencies #986
Comments
Is there anyone working on this? If not, I think I can do. Is this similar with this one kitze@cdaa892 |
@dvkndn 👍 |
Ok I'll make a PR for this one. All credit to @kitze |
@dvkndn finally. I'm not sure if that was causing this issue though, some people still have problems on specific verisons of npm 😥 |
Oh yeah, sorry for not commenting on this issue. I needed this for fixing the ESLint integration and forgot that there was a separate issue about this. |
Don't worry, I haven't started yet 😄 |
Fixed in #1068. |
We've had too many issues related to how the buggy npm client treats bundledDependencies.
Since LTS Node is now shipped with npm@3, we should remove bundledDependencies, bump npm version in engines field, and mention that we are npm@3+ in README.
The text was updated successfully, but these errors were encountered: