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

Update .npmignore to ignore .github and test folders #865

Merged
merged 1 commit into from
Jan 12, 2022

Conversation

SergeAstapov
Copy link
Contributor

This shrinks the "unpacked size" for a few Kb

Copy link
Member

@mansona mansona left a comment

Choose a reason for hiding this comment

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

So usually I would say that this sort of change isn't really that important since a) the "unpacked size" isn't all that representative of reality most of the time in the ember world and b) it matters less with something that is more of a server technology.

In this case there is literally no downside so I'm going to merge it 👍

@mansona mansona merged commit 193e13d into ember-fastboot:master Jan 12, 2022
@SergeAstapov SergeAstapov deleted the update-npmignore branch January 18, 2022 17:14
@SergeAstapov
Copy link
Contributor Author

@mansona totally agree with you! This is just to keep things tidy

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants