Skip to content
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

Add files entry to package.json. #26

Merged
merged 1 commit into from
Dec 21, 2016
Merged

Conversation

jamestalmage
Copy link
Contributor

Removes unnecessary files from the deployed package.

Deploying unnecessary files impacts the performance of npm install for everyone.

See: https://github.com/insin/package-config-checker

Removes unnecessary files from the deployed package.

Deploying unnecessary files impacts the performance of `npm install` for everyone.

See: https://github.com/insin/package-config-checker
@marcbachmann
Copy link
Collaborator

Shouldn't we also include the license file?
Do you think the license definition in the package.json is enough?

@jamestalmage
Copy link
Contributor Author

License files, and the Readme are included automatically (you can't disable, even if you wanted to).

@marcbachmann
Copy link
Collaborator

Ok. thanks.

@jvanharn
Copy link

Bump. @janl @marcbachmann
Could this be merged? Minimal difference, but would still be hugely appreciated.

@marcbachmann
Copy link
Collaborator

Yes, I'll do a release soon.

@marcbachmann marcbachmann merged commit 9bce5b3 into janl:master Dec 21, 2016
@marcbachmann
Copy link
Collaborator

Done. Released as 4.0.1

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

Successfully merging this pull request may close these issues.

3 participants