Skip to content

chore(.npmignore): update npmignore#339

Merged
nfischer merged 1 commit intomasterfrom
better-npmignore
Feb 4, 2016
Merged

chore(.npmignore): update npmignore#339
nfischer merged 1 commit intomasterfrom
better-npmignore

Conversation

@ariporad
Copy link
Copy Markdown
Contributor

@ariporad ariporad commented Feb 4, 2016

No description provided.

@ariporad
Copy link
Copy Markdown
Contributor Author

ariporad commented Feb 4, 2016

@nfischer: Ready for review.

.npmignore Outdated
test/
tmp/ No newline at end of file
tmp/
scripts/
Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

I think we can keep this

Copy link
Copy Markdown
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

@nfischer: I suppose we could, but I can't think of a reason anyone who downloads it from npm would ever need them, so I thought we may as well save them the bytes.

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

I guess the only reason is so that they can run tests to verify their build works. It might be helpful for debugging someone's problems

Copy link
Copy Markdown
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Hmm... Ok.

@ariporad
Copy link
Copy Markdown
Contributor Author

ariporad commented Feb 4, 2016

@nfischer: should be fixed now.

@nfischer
Copy link
Copy Markdown
Member

nfischer commented Feb 4, 2016

LGTM

nfischer added a commit that referenced this pull request Feb 4, 2016
chore(.npmignore): update npmignore
@nfischer nfischer merged commit 9ecff1f into master Feb 4, 2016
@nfischer nfischer deleted the better-npmignore branch February 4, 2016 23:32
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants