deps: upgrade to npm 2.14.16 #4960
Conversation
zkat
added
npm
|
CI: https://ci.nodejs.org/job/node-test-pull-request/1436/ It is worth mentioning the CI does not run |
|
Noted! If you want, |
|
Tests passing locally on OSX 10.10.5
|
|
I opened an issue nodejs/build#317 to discuss CI plans |
|
|
|
@Fishrock123 Next release (v2.14.17) will include the patch for pre-releases without cherry-picking btw. |
|
So I went ahead and made a CI job for npm and am doing a test run I'm expecting failures. To reduce noise I'll post updates in this comment |
|
@zkat this doesn't include that? I didn't modify Edit: can confirm I did indeed build the branch and instal before testing. |
|
@Fishrock123 Oh sorry, it wasn't clear: I included it as an extra patch cherry-picked from the last release, because it won't be included in npm itself until next week. See 24b8a5f |
|
LGTM |
PR-URL: nodejs#4960 Reviewed-By: Jeremiah Senkpiel <fishrock123@rocketmail.com> Reviewed-By: James M Snell <jasnell@gmail.com>
PR-URL: nodejs#4960 Reviewed-By: Jeremiah Senkpiel <fishrock123@rocketmail.com> Reviewed-By: James M Snell <jasnell@gmail.com>
PR-URL: #4960 Reviewed-By: Jeremiah Senkpiel <fishrock123@rocketmail.com> Reviewed-By: James M Snell <jasnell@gmail.com>
PR-URL: #4960 Reviewed-By: Jeremiah Senkpiel <fishrock123@rocketmail.com> Reviewed-By: James M Snell <jasnell@gmail.com>
|
Looks like these commits are on the staging branch. Closing landed in fcb8cae...9a9ac8e |
PR-URL: #4960 Reviewed-By: Jeremiah Senkpiel <fishrock123@rocketmail.com> Reviewed-By: James M Snell <jasnell@gmail.com>
PR-URL: #4960 Reviewed-By: Jeremiah Senkpiel <fishrock123@rocketmail.com> Reviewed-By: James M Snell <jasnell@gmail.com>
PR-URL: nodejs#4960 Reviewed-By: Jeremiah Senkpiel <fishrock123@rocketmail.com> Reviewed-By: James M Snell <jasnell@gmail.com>
PR-URL: nodejs#4960 Reviewed-By: Jeremiah Senkpiel <fishrock123@rocketmail.com> Reviewed-By: James M Snell <jasnell@gmail.com>
PR-URL: #4960 Reviewed-By: Jeremiah Senkpiel <fishrock123@rocketmail.com> Reviewed-By: James M Snell <jasnell@gmail.com>
PR-URL: #4960 Reviewed-By: Jeremiah Senkpiel <fishrock123@rocketmail.com> Reviewed-By: James M Snell <jasnell@gmail.com>
PR-URL: #4960 Reviewed-By: Jeremiah Senkpiel <fishrock123@rocketmail.com> Reviewed-By: James M Snell <jasnell@gmail.com>
PR-URL: #4960 Reviewed-By: Jeremiah Senkpiel <fishrock123@rocketmail.com> Reviewed-By: James M Snell <jasnell@gmail.com>
PR-URL: #4960 Reviewed-By: Jeremiah Senkpiel <fishrock123@rocketmail.com> Reviewed-By: James M Snell <jasnell@gmail.com>
PR-URL: #4960 Reviewed-By: Jeremiah Senkpiel <fishrock123@rocketmail.com> Reviewed-By: James M Snell <jasnell@gmail.com>
Another day, another LTS!
The changelog for this version can be found here: https://github.com/npm/npm/releases/tag/v2.14.16
Interesting changes:
24b13fb#11158 and5c0e4c4#10940 which include patches by @orangemocha that they had asked about in deps: upgrade to npm 2.14.15 #487264995be75ab216a9f6fe9649c19394cb05e6541690255be6f9e84fa48a587b0bf812a5#10903 Get rid of nock from tests, and get Travis green, which might hopefully also make CI in general more reliable. (@zkat and @iarna)r: @Fishrock123
r: @iarna