Skip to content
This repository was archived by the owner on Apr 22, 2023. It is now read-only.

deps: upgrade npm to 2.6.1 #9297

Closed
wants to merge 1 commit into from
Closed

deps: upgrade npm to 2.6.1 #9297

wants to merge 1 commit into from

Conversation

othiym23
Copy link

This is a comparatively modest update, at least compared to npm@2.7.0, which includes at least one feature that is very relevant to io.js's interests (foreshadowing!). Here are the pertinent changes in npm@2.6.1:

  • 8b98f0e
    #4471 npm outdated (and only npm outdated) now defaults to --depth=0. This also has the excellent but unexpected effect of making npm update -g work the way almost everyone wants it to. See the docs for
    --depth

    for the mildly confusing details. (@smikes)
  • aa79194
    #6565 Tweak peerDependency
    deprecation warning to include which peer dependency on which package is
    going to need to change. (@othiym23)
  • 5fa067f
    #7171 Tweak engineStrict
    deprecation warning to include which package.json is using it.
    (@othiym23)

This PR also includes all of the changes in npm@2.6.0:

  • 38c4825 #5068 Add new logout command, and make it do something useful on both bearer-based and basic-based authed clients. (@othiym23)
  • c8e08e6 #6565 Warn that peerDependency behavior is changing and add a note to the docs. (@othiym23)
  • 7c81a5f #7171 Warn that engineStrict in package.json will be going away in the next major version of npm (coming soon!) (@othiym23)

@misterdjules
Copy link

LGTM and make test-npm passes, thanks @othiym23!

@joyent/node-coreteam Any objection?

@misterdjules misterdjules added this to the 0.12.1 milestone Mar 3, 2015
@misterdjules misterdjules self-assigned this Mar 3, 2015
@cjihrig
Copy link

cjihrig commented Mar 3, 2015

LGTM

1 similar comment
@trevnorris
Copy link

LGTM

misterdjules pushed a commit to misterdjules/node that referenced this pull request Mar 3, 2015
PR: nodejs#9297
PR-URL: nodejs#9297
Reviewed-By: Julien Gilli <julien.gilli@joyent.com>
Reviewed-By: Trevor Norris <trev.norris@gmail.com>
Reviewed-By: Colin Ihrig <cjihrig@gmail.com>
@misterdjules
Copy link

Thank you! Landed in 63fbd10.

@othiym23
Copy link
Author

othiym23 commented Mar 3, 2015

Thank you, @misterdjules / everybody!

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

Successfully merging this pull request may close these issues.

6 participants