-
Notifications
You must be signed in to change notification settings - Fork 30k
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: upgrade npm to 4.1.1 #10781
deps: upgrade npm to 4.1.1 #10781
Conversation
Will be able to give this a more thorough review later on but just quickly on the file URL bit... once the new WHATWG URL stuff moves out of experimental, it's handling of |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
npm tests pass
Oops, forgot to merge this. D: Doing it now! |
Thanks, landed in f73bc91 |
PR-URL: #10781 Reviewed-By: Jeremiah Senkpiel <fishrock123@rocketmail.com>
PR-URL: #10781 Reviewed-By: Jeremiah Senkpiel <fishrock123@rocketmail.com>
PR-URL: nodejs#10781 Reviewed-By: Jeremiah Senkpiel <fishrock123@rocketmail.com>
PR-URL: nodejs#10781 Reviewed-By: Jeremiah Senkpiel <fishrock123@rocketmail.com>
Checklist
make -j8 test
(UNIX), orvcbuild test nosign
(Windows) passesAffected core subsystem(s)
deps
Description of change
npm doctor
command.Changelogs
v4.1.0
v4.1.1
r: @Fishrock123
r: @addaleax
r: @jasnell