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

[V3] Remove npm dependency #369

Closed
wants to merge 2 commits into from
Closed

Conversation

bastienmoulia
Copy link

@bastienmoulia bastienmoulia commented Jun 25, 2017

Linked to #350

@bastienmoulia bastienmoulia changed the title Remove npm dependency [V3] Remove npm dependency Jun 25, 2017
@yvele
Copy link

yvele commented Jul 12, 2017

Yep, having a local dependency to npm-check-updates makes having his npm v3 in nodes_modules/.bin, and makes npm scripts using it instead of the global npm.

Resulting in lot of bugs (exactly like parshap/check-node-version#12)

@vdh
Copy link

vdh commented Jul 19, 2017

It looks like npmi will also need to be removed? Otherwise its dependency to npm will still install the second copy.

@raineorshine
Copy link
Owner

Yes, npmi needs to be removed as well.

@maxleiko
Copy link

You guys can bump npmi to 4.0.0 in order to get rid of the npm 👍

@raineorshine
Copy link
Owner

Closing as the local npm has been removed in the soon-to-be-released v3.

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

Successfully merging this pull request may close these issues.

5 participants