-
-
Notifications
You must be signed in to change notification settings - Fork 292
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
npm package bundles node_modules? #651
Comments
More context here: #624 In short, it's needed to work around npm bugs. |
Probably better to bring this up on the npm issue tracker. |
3 tasks
Gotcha:
I tried summarizing on the npm issue tracker: npm/cli#4441 Thanks! |
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
I noticed that some versions' npm package bundles a node_modules directory, e.g.
Versions without node_modules are +/- 11 files and +/- 64 kB vs. versions with are +/- 1,800 files and +/- 6 MB.
I'm also getting an error installing an XO dependant with npm 7:
Don't know for sure if the bundled node_modules is the culprit, but it looked unintended. Is it?
I haven't figured out why it's there in some versions and not others. I'd assumed that
npm publish
would exclude node_modules. It is listed in .gitignore ...The text was updated successfully, but these errors were encountered: