Add `files` to `package.json`.
wtgtybhertgeghgtwtg opened this issue · 1 comments
This would prevent the test folder from being published to NPM. It'd shave like 40-some KB off the package. I can submit a PR if that is alright.
I personally don't think it's a good idea to ignore the test/ folder. That's documentation for the module that will now be missing when you're offline and need to look inside node_modules/.
Ignoring huge test files is a good idea, but ignoring a few text files won't make a meaningful difference. npm install is primarily slow because of latency and roundtrips, in my experience. I don't think this is the right tradeoff.
Previous discussions:
nodejs/readable-stream#100
rvagg/through2#36
browserify/browserify#1093
browserify/sha.js#5
feross/string-to-stream#1
beatgammit/base64-js#16
mafintosh/csv-parser#43
feross/buffer#98