-
Notifications
You must be signed in to change notification settings - Fork 30.7k
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 in LTS to 2.14.21 in Node.js 0.12 LTS #5594
Conversation
(The test failure recurs if I manually clear out that file:
) |
leave the LICENSE thing with me for now |
the results are in from citgm
OSX failed due to jenkins weirdness https://ci.nodejs.org/job/thealphanerd-smoker/108/nodes=osx1010/console I'm going to test locally |
jenkins weirdness should be resolved now |
Latest run: https://ci.nodejs.org/job/thealphanerd-smoker/109/ edit: the moment failure we are seeing on ubuntu is being tracked up stream |
WRT the moment.js error, it's DST related, and should go away on its own by this Monday. We have a fix pending in a PR, but see no need to push a hotfix release for such a minor issue at this time. |
Could this be closed now? |
We have a newer version of npm in v0.12 now. closing |
Pull Request check-list
Please make sure to review and check all of these items:
make -j8 test
(UNIX) orvcbuild test nosign
(Windows) pass withthis change (including linting)? (see below)
Affected core subsystem(s)
deps
Description of change
Upgrades the v0.12 LTS branch to use
npm@lts
/npm@2
.Got one repeatable test failure, but am confident it's not related to this change:
We've also confirmed that npm's tests run when run against Node 0.12 in Travis. It should also get a
citgm
run.I tried to be conservative about what tags I applied, and apologize in advance if I misinterpreted the meanings of the ones that I set. This might also need
lts-agenda
, since I understand this will be discussed at this week's LTS meeting.[email protected]
includes the updatedLICENSE
which includes the removal of the potentially confusing web site and registry Terms of Use. v0.12 (also) doesn't have the license updating tool, so is there a best practice for updating the top-levelLICENSE
to reflect the changed npm license?r: @thealphanerd
r: @Fishrock123