FazBrowse GitHub Viewer | Trending |
URL:
| Home
Tools: [Download Repo ZIP]   [Original HTTPS Page]

deps: upgrade npm to 5.7.1 by targos · Pull Request #19302 · nodejs/node · GitHub

/ node Public

deps: upgrade npm to 5.7.1 - #19302

Closed
targos wants to merge 1 commit into
nodejs:masterfrom
targos:npm-5.7.1
Closed

deps: upgrade npm to 5.7.1#19302
targos wants to merge 1 commit into
nodejs:masterfrom
targos:npm-5.7.1

Conversation

targos commented Mar 12, 2018

Copy link
Copy Markdown
Member

Depends on #19293

Checklist
  • make -j4 test (UNIX), or vcbuild test (Windows) passes
  • commit message follows commit guidelines

targos added the blocked PRs that are blocked by other issues or PRs. label Mar 12, 2018
nodejs-github-bot added the npm Issues and PRs related to the npm client dependency or the npm registry. label Mar 12, 2018

cjihrig commented Mar 12, 2018

Copy link
Copy Markdown
Contributor

How does this relate to #19298? The diffs are pretty different in size.

Copy link
Copy Markdown
Contributor

it's possible that #19298 didn't follow the upgrade guide, in turn didn't whitespace fix?

targos commented Mar 12, 2018

Copy link
Copy Markdown
Member Author

Whoops, I missed #19298. Let's close and check there what happened.

targos closed this Mar 12, 2018
targos deleted the npm-5.7.1 branch April 19, 2018 13:38
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters. Learn more about bidirectional Unicode characters
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

blocked PRs that are blocked by other issues or PRs. 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


Back | FazBrowse Home | New Git URL