InstantJasmineCoffee icon indicating copy to clipboard operation
InstantJasmineCoffee copied to clipboard

Implement package.json and remove node_modules folder from repository

Open Ricket opened this issue 13 years ago • 1 comments

Just because NPM 1.0 separated global and local packages doesn't mean you should check node modules into your repository. You should add a package.json file describing the dependencies and remove the node_modules folder from the repository. Then, when an user clones the repository they should cd into the folder and run npm install to install the latest versions of the necessary modules to their local space.

http://npmjs.org/doc/json.html

You can find examples of this in other CoffeeScript projects.

Ricket avatar Jul 08 '12 03:07 Ricket

Hmm, yes, that is clearly the right method. Though at the time, more than a year ago, all I had to go off of was the post that I link to in the README. It doesn't mention package.json so I wonder if it was available in that RC?

Regardless, I agree that it's silly to have committed versions of required modules here. Perhaps I will get around to making that fix but I must be honest it is not high on my list of chores ATM. Pull req welcome. :wink:

krismolendyke avatar Aug 28 '12 22:08 krismolendyke