Skip to content

deps: upgrade npm to 2.7.3

Rodrigo Muino Tomonari requested to merge github/fork/npm/npm-2.7.3 into v1.x

Since I'm now an io.js collaborator, I'm happy to merge this myself. However, I still need sign-off from one or more other collaborators, so give this a look if you have a few minutes.

This is another bug release, and this is actually a roll-up of two versions, because an interim release was made to the canary release to fix a nasty regression that had crept in:

  • 1549106 #7641 Due to 448efd0, running npm shrinkwrap --dev caused production dependencies to no longer be included in npm-shrinkwrap.json. Whoopsie! (@othiym23)

Aside from that, here are the two most important fixes from 2.7.2:

  • fb0ac26 #7579 Only block removing files and links when we're sure npm isn't responsible for them. This change is hard to summarize, because if things are working correctly you should never see it, but if you want more context, just go read the commit message, which lays it all out. (@othiym23)
  • 051c473 #7552 bundledDependencies are now properly included in the installation context. This is another fantastically hard-to-summarize bug, and once again, I encourage you to read the commit message if you're curious about the details. The snappy takeaway is that this unbreaks many use cases for ember-cli. (@othiym23)

There are, as usual, a few other smaller fixes in the release as well. I've applied the floating patch for node-gyp yet again, and would very much like to not have to do this for too many more releases. Please?

R=@rvagg R=@Fishrock123

Merge request reports

Loading