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 npmignore file. #20

Merged
merged 4 commits into from
Apr 30, 2016
Merged

Add npmignore file. #20

merged 4 commits into from
Apr 30, 2016

Conversation

rog
Copy link
Collaborator

@rog rog commented Apr 28, 2016

The devDependencies section of npm's package.json documentation says to list your test dependencies there so that users of your package don't have to pull down extra dependencies.

A .npmignore file works in similar way so users don't have to download the test and the examples folders when they install paperpress from npm install paperpress, only download the library.

@coveralls
Copy link

coveralls commented Apr 29, 2016

Coverage Status

Coverage remained the same at 100.0% when pulling d96df4c on npmignore into df4086b on master.

@Siedrix Siedrix merged commit c01f7c9 into master Apr 30, 2016
@rog rog deleted the npmignore branch May 1, 2016 06:30
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.

None yet

3 participants