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

deps: update npm to 2.13.1 #2210

Closed
wants to merge 3 commits into from
Closed

deps: update npm to 2.13.1 #2210

wants to merge 3 commits into from

Conversation

zkat
Copy link
Contributor

@zkat zkat commented Jul 21, 2015

Hey! Here's a new npm for you. We figured you'd rather get it on Monday than Friday. :) You can check out the release notes here: https://github.com/npm/npm/releases/tag/v2.13.1

Mostly minor bugfix and internal stuff (docs, deps, refactor, test) bits.

r: @Fishrock123
r: @chrisdickinson

zkat and others added 3 commits July 20, 2015 20:07
Every npm version bump requires a few patches to be floated on
node-gyp for io.js compatibility. These patches are found in
03d1992,
5de334c, and
da730c7. This commit squashes
them into a single commit.

PR-URL: nodejs#990
Reviewed-By: Ben Noordhuis <info@bnoordhuis.nl>
The delay-load hook allows node.exe/iojs.exe to be renamed. See efadffe
for more background.

PR-URL: nodejs#1433
Reviewed-By: Ben Noordhuis <info@bnoordhuis.nl>
@zkat zkat changed the title deps: npm 2.13.1 deps: update npm to 2.13.1 Jul 21, 2015
@Fishrock123
Copy link
Contributor

We figured you'd rather get it on Monday than Friday.

Actually, Friday is probably better since it'l get in sooner (if you have the time on fridays), but it doesn't matter too much to me. (since I almost exclusively review/land them right now)

@zkat
Copy link
Contributor Author

zkat commented Jul 21, 2015

Noted! Thanks for your diligence as usual, Jeremiah <3

@Fishrock123
Copy link
Contributor

Tests are passing, will skim the code and merge tomorrow.

@Fishrock123 Fishrock123 added the npm Issues and PRs related to the npm client dependency or the npm registry. label Jul 21, 2015
Fishrock123 pushed a commit that referenced this pull request Jul 21, 2015
PR-URL: #2210
Reviewed-By: Jeremiah Senkpiel <fishrock123@rocketmail.com>
@Fishrock123
Copy link
Contributor

Thanks, landed in 6391f4d...f90f1e7 :D

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
npm Issues and PRs related to the npm client dependency or the npm registry.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants