-
Notifications
You must be signed in to change notification settings - Fork 0
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
Version 3.0.0 of the module does not include the required node_modules #3
Comments
Also running into this issue, would love to see it fixed. |
Please try the beta release here and let me know if that resolves your issue |
I downloaded the beta. It is well over 40Mb unpacked and still includes large libraries like React, React Native and the react devtools. Pruning non-JS stuff is a step but not enough. For NativeFileDocuments, I have removed any library that turned out to be dev dependency. I ship only 11 modules with the module, it would have been over 700 otherwise. Same here. The library does not need all these modules to work, these are dev dependencies. |
I'm trying to find a middle ground between including no dependencies and letting the developer figure it out versus providing what's needed, without spending hours manually combing through dependencies. The script is a huge help. Do you have any other suggestions for automated ways to remove the extra dev dependencies? For NativeFileDocuments, did you just manually traverse the package.json files? |
For NativeFileDocuments, I moved any modules except the ones I installed away to a temp folder. Then I tested and moved back modules until I got no errors anymore. But I think there are libraries or even just NPM commands that let you weed out the dev dependencies. |
interestingly, the |
will try with |
Would be interesting if that works. I will then try to see how that works on NativeFileDocuments |
after {
"added": 0,
"removed": 0,
"changed": 44,
"audited": 723,
"funding": 33,
"audit": {
"vulnerabilities": {
"info": 0,
"low": 0,
"moderate": 6,
"high": 2,
"critical": 0,
"total": 8
},
"dependencies": {
"prod": 77,
"dev": 0,
"optional": 0,
"peer": 646,
"peerOptional": 0,
"total": 722
}
}
} |
tried removing |
I ran |
Version 3.0.0 of the module does not include the required node_modules.
Running npm install inside the module folder in javascriptsources could fix this but this brings in 700+ folders with well over 200Mb of files. Way too much to check in.
Deployment packages created with this release will also fail because the packager will not have these files as well.
Please include the node modules for the minimum required modules.
The text was updated successfully, but these errors were encountered: