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

Shouldn't check in node_modules to version control #1

Open
spikebrehm opened this issue Dec 9, 2013 · 3 comments
Open

Shouldn't check in node_modules to version control #1

spikebrehm opened this issue Dec 9, 2013 · 3 comments

Comments

@spikebrehm
Copy link

Standard practice is to add it to your .gitignore.

@smclelland
Copy link

Yep.  mistake.  will fix.

-- 
Steve McLelland
Sent with Airmail

On December 9, 2013 at 1:25:41 PM, Spike Brehm ([email protected]) wrote:

Standard practice is to add it to your `.gitignore.


Reply to this email directly or view it on GitHub.

@kylev
Copy link
Contributor

kylev commented Jan 24, 2014

My request is essentially git rm -r node_modules.

It seems that you may also need to add handlebars to the package.json, but I'm not 100% on that. The mocha tests won't run without it.

@kylev
Copy link
Contributor

kylev commented Jun 23, 2016

Pretty rare that a 556 file change is still merge-able after 2 years. Merge it?

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

No branches or pull requests

3 participants