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.

Sign up for GitHub

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

Jump to bottom

deps: upgrade npm to 7.24.0 #40167

Closed
wants to merge 1 commit into from
Closed

deps: upgrade npm to 7.24.0 #40167

wants to merge 1 commit into from

Conversation

npm-robot
Copy link
Contributor

v7.24.0 (2021-09-16)

FEATURES

BUG FIXES

DEPENDENCIES

  • ac8e4ad18 init-package-json@2.0.5:
    • fix: bin script path
  • 371655a6b minipass@3.1.5:
    • fix: re-emit 'error' event if missed and new listener added
    • fix: do not blow up if process is missing

DOCUMENTATION

@nodejs-github-bot nodejs-github-bot added dont-land-on-v12.x fast-track PRs that do not need to wait for 48 hours to land. needs-ci PRs that need a full CI run. npm Issues and PRs related to the npm client dependency or the npm registry. labels Sep 20, 2021
@github-actions
Copy link
Contributor

Fast-track has been requested by @nodejs-github-bot. Please 👍 to approve.

@nodejs-github-bot
Copy link
Collaborator

CI: https://ci.nodejs.org/job/node-test-pull-request/40004/

@nodejs-github-bot
Copy link
Collaborator

CI: https://ci.nodejs.org/job/node-test-pull-request/40005/

@nodejs-github-bot
Copy link
Collaborator

CI: https://ci.nodejs.org/job/node-test-pull-request/40006/

@nodejs-github-bot
Copy link
Collaborator

CI: https://ci.nodejs.org/job/node-test-pull-request/40007/

@Trott Trott added the commit-queue Add this label to land a pull request using GitHub Actions. label Sep 20, 2021
@github-actions github-actions bot removed the commit-queue Add this label to land a pull request using GitHub Actions. label Sep 20, 2021
@github-actions
Copy link
Contributor

Commit Queue failed
- Loading data for nodejs/node/pull/40167
✔  Done loading data for nodejs/node/pull/40167
----------------------------------- PR info ------------------------------------
Title      deps: upgrade npm to 7.24.0 (#40167)
   ⚠  Could not retrieve the email or name of the PR author's from user's GitHub profile!
Branch     npm-robot:npm-7.24.0 -> nodejs:master
Labels     npm, fast-track, needs-ci, dont-land-on-v12.x, dont-land-on-v14.x
Commits    1
 - deps: upgrade npm to 7.24.0
Committers 1
 - npm team 
PR-URL: https://github.com/nodejs/node/pull/40167
Fixes: https://github.com/bin
Fixes: https://github.com/re-emit
Fixes: https://github.com/do
Reviewed-By: Myles Borins 
Reviewed-By: Rich Trott 
------------------------------ Generated metadata ------------------------------
PR-URL: https://github.com/nodejs/node/pull/40167
Fixes: https://github.com/bin
Fixes: https://github.com/re-emit
Fixes: https://github.com/do
Reviewed-By: Myles Borins 
Reviewed-By: Rich Trott 
--------------------------------------------------------------------------------
   ℹ  This PR was created on Mon, 20 Sep 2021 19:19:00 GMT
   ✔  Approvals: 2
   ✔  - Myles Borins (@MylesBorins) (TSC): https://github.com/nodejs/node/pull/40167#pullrequestreview-759019696
   ✔  - Rich Trott (@Trott) (TSC): https://github.com/nodejs/node/pull/40167#pullrequestreview-759069159
   ℹ  This PR is being fast-tracked
   ✔  Last GitHub Actions successful
   ℹ  Last Full PR CI on 2021-09-20T22:01:24Z: https://ci.nodejs.org/job/node-test-pull-request/40007/
- Querying data for job/node-test-pull-request/40007/
✔  Build data downloaded
   ✔  Last Jenkins CI successful
--------------------------------------------------------------------------------
   ✔  No git cherry-pick in progress
   ✔  No git am in progress
   ✔  No git rebase in progress
--------------------------------------------------------------------------------
- Bringing origin/master up to date...
From https://github.com/nodejs/node
 * branch                  master     -> FETCH_HEAD
✔  origin/master is now up-to-date
- Downloading patch for 40167
From https://github.com/nodejs/node
 * branch                  refs/pull/40167/merge -> FETCH_HEAD
✔  Fetched commits as 931abc085e77..2d7087bae3fa
--------------------------------------------------------------------------------
Removing deps/npm/node_modules/init-package-json/LICENSE
[master 84d7d59b67] deps: upgrade npm to 7.24.0
 Author: npm team 
 Date: Mon Sep 20 19:18:58 2021 +0000
 28 files changed, 442 insertions(+), 194 deletions(-)
 delete mode 100644 deps/npm/node_modules/init-package-json/LICENSE
 create mode 100644 deps/npm/node_modules/init-package-json/LICENSE.md
 rename deps/npm/node_modules/init-package-json/{ => lib}/default-input.js (64%)
 rename deps/npm/node_modules/init-package-json/{ => lib}/init-package-json.js (79%)
 rewrite deps/npm/test/lib/utils/did-you-mean.js (74%)
   ✔  Patches applied
--------------------------------------------------------------------------------
--------------------------------- New Message ----------------------------------
deps: upgrade npm to 7.24.0

PR-URL: #40167

Fixes: https://github.com/bin

Fixes: https://github.com/re-emit

Fixes: https://github.com/do

Reviewed-By: Myles Borins myles.borins@gmail.com

Reviewed-By: Rich Trott rtrott@gmail.com


[master 033dddab08] deps: upgrade npm to 7.24.0

Author: npm team ops+robot@npmjs.com

Date: Mon Sep 20 19:18:58 2021 +0000

28 files changed, 442 insertions(+), 194 deletions(-)

delete mode 100644 deps/npm/node_modules/init-package-json/LICENSE

create mode 100644 deps/npm/node_modules/init-package-json/LICENSE.md

rename deps/npm/node_modules/init-package-json/{ => lib}/default-input.js (64%)

rename deps/npm/node_modules/init-package-json/{ => lib}/init-package-json.js (79%)

rewrite deps/npm/test/lib/utils/did-you-mean.js (74%)

✖  033dddab08b6ea3d099f8f4280ca39720b3bef5c

✖  2:7      Fixes must be a GitHub URL.               fixes-url

✖  3:7      Fixes must be a GitHub URL.               fixes-url

✖  4:7      Fixes must be a GitHub URL.               fixes-url

✔  0:0      blank line after title                    line-after-title

✔  0:0      line-lengths are valid                    line-length

✔  0:0      metadata is at end of message             metadata-end

✔  1:8      PR-URL is valid.                          pr-url

✔  0:0      reviewers are valid                       reviewers

✔  0:0      valid subsystems                          subsystem

✔  0:0      Title is formatted correctly.             title-format

✔  0:0      Title is <= 50 columns.                   title-length

ℹ  Please fix the commit message and try again.
https://github.com/nodejs/node/actions/runs/1255430498

@github-actions github-actions bot added the commit-queue-failed An error occurred while landing this pull request using GitHub Actions. label Sep 20, 2021
@Trott
Copy link
Member

Trott commented Sep 20, 2021

Landed in 7153d25

Trott pushed a commit to Trott/io.js that referenced this pull request Sep 20, 2021
@npm-robot @Trott
deps: upgrade npm to 7.24.0
7153d25 
PR-URL: nodejs#40167
Reviewed-By: Myles Borins <myles.borins@gmail.com>
Reviewed-By: Rich Trott <rtrott@gmail.com>
@Trott Trott closed this Sep 20, 2021
BethGriggs pushed a commit that referenced this pull request Sep 21, 2021
@npm-robot @BethGriggs
deps: upgrade npm to 7.24.0
1ed4b37 
PR-URL: #40167
Reviewed-By: Myles Borins <myles.borins@gmail.com>
Reviewed-By: Rich Trott <rtrott@gmail.com>
BethGriggs pushed a commit that referenced this pull request Sep 21, 2021
@npm-robot @BethGriggs
deps: upgrade npm to 7.24.0
85206b7 
PR-URL: #40167
Reviewed-By: Myles Borins <myles.borins@gmail.com>
Reviewed-By: Rich Trott <rtrott@gmail.com>
@BethGriggs BethGriggs mentioned this pull request Sep 21, 2021
Merged
1 task
This was referenced Sep 22, 2021
Open
Open
Open
Open
Open
Open
Open
@github-actions github-actions bot mentioned this pull request Sep 29, 2021
Open
18 tasks
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@MylesBorins MylesBorins MylesBorins approved these changes

@Trott Trott Trott approved these changes

Assignees
No one assigned
Labels
commit-queue-failed An error occurred while landing this pull request using GitHub Actions. fast-track PRs that do not need to wait for 48 hours to land. needs-ci PRs that need a full CI run. npm Issues and PRs related to the npm client dependency or the npm registry.
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
4 participants
@npm-robot @nodejs-github-bot @Trott @MylesBorins