Skip to content

Reduce installed size #622

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

Closed
max-mapper opened this issue Nov 28, 2016 · 4 comments
Closed

Reduce installed size #622

max-mapper opened this issue Nov 28, 2016 · 4 comments

Comments

@max-mapper
Copy link

Hi, I noticed the installed size (of dependencies only, not devDependencies) is 410mb.

screen shot 2016-11-28 at 12 01 26 pm

It looks like ~%80 is coming from babel related modules (which I don't personally use). Is there any way to possibly make these optional, or have a lighter weight distribution in general?

Loving the module so far, thanks!

@jfirebaugh
Copy link
Member

Previously: #583. Are you using npm v2?

@max-mapper
Copy link
Author

Yep. Sorry, searched issues and didnt find that. I guess the fix then is for me to use something that dedupes babel modules.

@tmcw
Copy link
Member

tmcw commented Nov 28, 2016

Hey Max! Good to hear from ya

Yep, as @jfirebaugh, npm v2 is the cause of much of our installation troubles and will cause those troubles unavoidable on practically any module that uses Babel.

That said: even with npm v3, size is still an issue and one that we'd love to pare down, so that it's a no-brainer install even for those with very minimal requirements. So reducing size would be very welcome and I'm open to ideas. We use babylon as the parser central to documentation.js, so some reliance on the babel ecosystem would be hard to work away from - supporting a different parser would mean supporting a different AST format, so it might be a door to more complexity rather than less.

Besides Babel, we also just need to keep track of which dependencies are consuming space in node_modules and whether they're pulling their weight.

  • Tom

@tmcw
Copy link
Member

tmcw commented Dec 23, 2016

Closing - happy to accept PRs implementing any significant install size improvements. The problem encountered here is with npm v2, though, so it's not something I can fix.

@tmcw tmcw closed this as completed Dec 23, 2016
tmcw added a commit that referenced this issue Dec 23, 2016
tmcw added a commit that referenced this issue Dec 23, 2016
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

No branches or pull requests

3 participants