search

qunitjs / node-qunit

QUnit runner for Node.js.
MIT License
172 stars 53 forks source link

Consider adding .npmignore file #103

Closed hakubo closed 9 years ago

hakubo commented 10 years ago

Hey,

I am a happy user of your library, but as my project grows we have a lot of dependencies. We could save a lot of space if you added .npmignore file to your project.

I even found tool that can help you with it: https://github.com/inikulin/dmn

Thanks!

fyockm commented 10 years ago

@hakubo specifically- you don't want the test directory in your node_modules?

kof commented 10 years ago

I am not sure which files and folders need to be ignored.

  1. I never run into the issue with space
  2. Sometimes its convenient to read docs or run tests directly in the installed package
  3. Some packages even run some tests after installation
kof commented 10 years ago

maybe we need a tool to clean up packages after installation for this ... f.e. when installing in production env.

kof commented 9 years ago

dunno what to do here, @hakubo feel free to reopen if you want to share more insights.