node-hashring icon indicating copy to clipboard operation
node-hashring copied to clipboard

Ignore tests dir when publishing

Open cesine opened this issue 6 years ago • 0 comments

We noticed the test fixtures were in our production docker container

$ du -k node_modules/* | awk '$1 > 2000' | sort -nr # 2MB
4700  node_modules/hashring
4608  node_modules/@shutterstock
4596  node_modules/hashring/tests
4576  node_modules/hashring/tests/fixtures

Currently npm publish is using your .gitignore since there is no .npmignore provided https://docs.npmjs.com/misc/developers#keeping-files-out-of-your-package

This pr adds an .npmignore (based on your .gitignore to keep the existing functionality, with the tests directory excluded)

cesine avatar Dec 03 '19 16:12 cesine