Skip to content
This repository was archived by the owner on Jul 1, 2026. It is now read-only.

Reduce the size of the NPM distribution by ignoring unneeded files - #253

Closed
maxtaco wants to merge 1 commit into
TryGhost:masterfrom
maxtaco:master
Closed

Reduce the size of the NPM distribution by ignoring unneeded files#253
maxtaco wants to merge 1 commit into
TryGhost:masterfrom
maxtaco:master

Conversation

@maxtaco

@maxtaco maxtaco commented Feb 6, 2014

Copy link
Copy Markdown

The files in the deps/, test/ and benchmark/ directories are quite large, and as far as I can tell, aren't used once the code is packaged for npm. They can be left out of the npm tarball with this addition.

@kirkov

kirkov commented Feb 11, 2014

Copy link
Copy Markdown

also, the node-pre-gyp has a dependency on aws-sdk which takes up 15 MB after unpack. I've opened an issue here: aws/aws-sdk-js#233

@springmeyer

Copy link
Copy Markdown
Contributor

@maxtaco - thanks for the suggestion. I'll add this shortly not going to accept your pull because ./deps is need for a source compile.
@kirkov: aws size is handled by mapbox/node-pre-gyp#38.

springmeyer pushed a commit that referenced this pull request Feb 25, 2014
springmeyer pushed a commit that referenced this pull request Apr 9, 2014
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants