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.

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 8.2.0 #41065

Closed
wants to merge 1 commit into from
Closed

deps: upgrade npm to 8.2.0 #41065

wants to merge 1 commit into from

Conversation

npm-robot
Copy link
Contributor

v8.2.0 (2021-12-02)

Features

Bug Fixes

Documentation

Dependencies

@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 Dec 2, 2021
@github-actions
Copy link
Contributor

github-actions bot commented Dec 2, 2021

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

@wraithgar
Copy link

Another larger-than-normal diff on this one because we have refactored how logging works, namely by separating out terminal and file logging to their own concerns, and streaming to a logfile that is now always written even if npm exited cleanly.

@lpinca lpinca added the request-ci Add this label to start a Jenkins CI on a PR. label Dec 3, 2021
@github-actions github-actions bot removed the request-ci Add this label to start a Jenkins CI on a PR. label Dec 3, 2021
@nodejs-github-bot
Copy link
Collaborator

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

@github-actions github-actions bot mentioned this pull request Dec 4, 2021
Open
32 tasks
@nodejs-github-bot
Copy link
Collaborator

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

@github-actions github-actions bot mentioned this pull request Dec 5, 2021
Open
20 tasks
Trott
Trott approved these changes Dec 5, 2021
View reviewed changes
Copy link
Member

@Trott Trott left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Rubber stamp LGTM

@targos targos added the commit-queue Add this label to land a pull request using GitHub Actions. label Dec 5, 2021
@nodejs-github-bot nodejs-github-bot added commit-queue-failed An error occurred while landing this pull request using GitHub Actions. and removed commit-queue Add this label to land a pull request using GitHub Actions. labels Dec 5, 2021
@nodejs-github-bot
Copy link
Collaborator

Commit Queue failed
- Loading data for nodejs/node/pull/41065
✔  Done loading data for nodejs/node/pull/41065
----------------------------------- PR info ------------------------------------
Title      deps: upgrade npm to 8.2.0 (#41065)
   ⚠  Could not retrieve the email or name of the PR author's from user's GitHub profile!
Branch     npm-robot:npm-8.2.0 -> nodejs:master
Labels     npm, fast-track, needs-ci, dont-land-on-v12.x, dont-land-on-v14.x, lts-watch-v16.x
Commits    1
 - deps: upgrade npm to 8.2.0
Committers 1
 - npm team 
PR-URL: https://github.com/nodejs/node/pull/41065
Fixes: https://github.com/add
Fixes: https://github.com/always
Reviewed-By: Myles Borins 
Reviewed-By: Luigi Pinca 
Reviewed-By: Rich Trott 
------------------------------ Generated metadata ------------------------------
PR-URL: https://github.com/nodejs/node/pull/41065
Fixes: https://github.com/add
Fixes: https://github.com/always
Reviewed-By: Myles Borins 
Reviewed-By: Luigi Pinca 
Reviewed-By: Rich Trott 
--------------------------------------------------------------------------------
   ℹ  This PR was created on Thu, 02 Dec 2021 22:04:48 GMT
   ✔  Approvals: 3
   ✔  - Myles Borins (@MylesBorins) (TSC): https://github.com/nodejs/node/pull/41065#pullrequestreview-822168572
   ✔  - Luigi Pinca (@lpinca): https://github.com/nodejs/node/pull/41065#pullrequestreview-822428987
   ✔  - Rich Trott (@Trott) (TSC): https://github.com/nodejs/node/pull/41065#pullrequestreview-823349691
   ℹ  This PR is being fast-tracked
   ✔  Last GitHub Actions successful
   ℹ  Last Full PR CI on 2021-12-04T16:49:27Z: https://ci.nodejs.org/job/node-test-pull-request/41334/
- Querying data for job/node-test-pull-request/41334/
   ✔  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
   147d23b211..6ef6bdf1fd  master     -> origin/master
✔  origin/master is now up-to-date
master is out of sync with origin/master. Mismatched commits:
 - 3966111212 build: add `--without-corepack`
 - 6ef6bdf1fd build: add `--without-corepack`
--------------------------------------------------------------------------------
HEAD is now at 6ef6bdf1fd build: add `--without-corepack`
   ✔  Reset to origin/master
- Downloading patch for 41065
From https://github.com/nodejs/node
 * branch                  refs/pull/41065/merge -> FETCH_HEAD
✔  Fetched commits as 6ef6bdf1fd19..c2b5c0d7525c
--------------------------------------------------------------------------------
[master 1f36fd9896] deps: upgrade npm to 8.2.0
 Author: npm team 
 Date: Thu Dec 2 22:04:46 2021 +0000
 287 files changed, 5497 insertions(+), 5996 deletions(-)
 create mode 100644 deps/npm/docs/content/using-npm/logging.md
 create mode 100644 deps/npm/docs/output/using-npm/logging.html
 delete mode 100644 deps/npm/lib/utils/cleanup-log-files.js
 create mode 100644 deps/npm/lib/utils/display.js
 delete mode 100644 deps/npm/lib/utils/get-project-scope.js
 create mode 100644 deps/npm/lib/utils/log-file.js
 create mode 100644 deps/npm/lib/utils/log-shim.js
 delete mode 100644 deps/npm/lib/utils/proc-log-listener.js
 delete mode 100644 deps/npm/lib/utils/setup-log.js
 create mode 100644 deps/npm/lib/utils/timers.js
 create mode 100644 deps/npm/lib/utils/with-chown-sync.js
 create mode 100644 deps/npm/man/man7/logging.7
 delete mode 100644 deps/npm/node_modules/code-point-at/index.js
 delete mode 100644 deps/npm/node_modules/code-point-at/license
 delete mode 100644 deps/npm/node_modules/code-point-at/package.json
 delete mode 100644 deps/npm/node_modules/code-point-at/readme.md
 delete mode 100644 deps/npm/node_modules/node-gyp/node_modules/aproba/LICENSE
 delete mode 100644 deps/npm/node_modules/node-gyp/node_modules/aproba/index.js
 delete mode 100644 deps/npm/node_modules/node-gyp/node_modules/aproba/package.json
 delete mode 100644 deps/npm/node_modules/node-gyp/node_modules/gauge/LICENSE
 delete mode 100644 deps/npm/node_modules/node-gyp/node_modules/gauge/base-theme.js
 delete mode 100644 deps/npm/node_modules/node-gyp/node_modules/gauge/error.js
 delete mode 100644 deps/npm/node_modules/node-gyp/node_modules/gauge/has-color.js
 delete mode 100644 deps/npm/node_modules/node-gyp/node_modules/gauge/index.js
 delete mode 100644 deps/npm/node_modules/node-gyp/node_modules/gauge/package.json
 delete mode 100644 deps/npm/node_modules/node-gyp/node_modules/gauge/plumbing.js
 delete mode 100644 deps/npm/node_modules/node-gyp/node_modules/gauge/process.js
 delete mode 100644 deps/npm/node_modules/node-gyp/node_modules/gauge/progress-bar.js
 delete mode 100644 deps/npm/node_modules/node-gyp/node_modules/gauge/render-template.js
 delete mode 100644 deps/npm/node_modules/node-gyp/node_modules/gauge/set-immediate.js
 delete mode 100644 deps/npm/node_modules/node-gyp/node_modules/gauge/set-interval.js
 delete mode 100644 deps/npm/node_modules/node-gyp/node_modules/gauge/spin.js
 delete mode 100644 deps/npm/node_modules/node-gyp/node_modules/gauge/template-item.js
 delete mode 100644 deps/npm/node_modules/node-gyp/node_modules/gauge/theme-set.js
 delete mode 100644 deps/npm/node_modules/node-gyp/node_modules/gauge/themes.js
 delete mode 100644 deps/npm/node_modules/node-gyp/node_modules/gauge/wide-truncate.js
 delete mode 100644 deps/npm/node_modules/node-gyp/node_modules/is-fullwidth-code-point/index.js
 delete mode 100644 deps/npm/node_modules/node-gyp/node_modules/is-fullwidth-code-point/license
 delete mode 100644 deps/npm/node_modules/node-gyp/node_modules/is-fullwidth-code-point/package.json
 delete mode 100644 deps/npm/node_modules/node-gyp/node_modules/npmlog/LICENSE
 delete mode 100644 deps/npm/node_modules/node-gyp/node_modules/npmlog/log.js
 delete mode 100644 deps/npm/node_modules/node-gyp/node_modules/npmlog/package.json
 delete mode 100644 deps/npm/node_modules/node-gyp/node_modules/string-width/index.js
 delete mode 100644 deps/npm/node_modules/node-gyp/node_modules/string-width/license
 delete mode 100644 deps/npm/node_modules/node-gyp/node_modules/string-width/package.json
 delete mode 100644 deps/npm/node_modules/npmlog/node_modules/are-we-there-yet/LICENSE.md
 delete mode 100644 deps/npm/node_modules/npmlog/node_modules/are-we-there-yet/lib/index.js
 delete mode 100644 deps/npm/node_modules/npmlog/node_modules/are-we-there-yet/lib/tracker-base.js
 delete mode 100644 deps/npm/node_modules/npmlog/node_modules/are-we-there-yet/lib/tracker-group.js
 delete mode 100644 deps/npm/node_modules/npmlog/node_modules/are-we-there-yet/lib/tracker-stream.js
 delete mode 100644 deps/npm/node_modules/npmlog/node_modules/are-we-there-yet/lib/tracker.js
 delete mode 100644 deps/npm/node_modules/npmlog/node_modules/are-we-there-yet/package.json
 delete mode 100644 deps/npm/node_modules/number-is-nan/index.js
 delete mode 100644 deps/npm/node_modules/number-is-nan/license
 delete mode 100644 deps/npm/node_modules/number-is-nan/package.json
 delete mode 100644 deps/npm/node_modules/number-is-nan/readme.md
 create mode 100644 deps/npm/tap-snapshots/test/lib/utils/log-file.js.test.cjs
 create mode 100644 deps/npm/test/fixtures/clean-snapshot.js
 create mode 100644 deps/npm/test/fixtures/mock-globals.js
 create mode 100644 deps/npm/test/fixtures/mock-logs.js
 rewrite deps/npm/test/fixtures/mock-npm.js (65%)
 rewrite deps/npm/test/lib/cli.js (63%)
 rewrite deps/npm/test/lib/commands/dedupe.js (69%)
 rewrite deps/npm/test/lib/commands/find-dupes.js (80%)
 rewrite deps/npm/test/lib/commands/prune.js (75%)
 rewrite deps/npm/test/lib/commands/version.js (82%)
 create mode 100644 deps/npm/test/lib/fixtures/mock-globals.js
 rewrite deps/npm/test/lib/load-all.js (70%)
 delete mode 100644 deps/npm/test/lib/utils/cleanup-log-files.js
 create mode 100644 deps/npm/test/lib/utils/display.js
 rewrite deps/npm/test/lib/utils/exit-handler.js (81%)
 delete mode 100644 deps/npm/test/lib/utils/get-project-scope.js
 rewrite deps/npm/test/lib/utils/is-windows-bash.js (65%)
 create mode 100644 deps/npm/test/lib/utils/log-file.js
 create mode 100644 deps/npm/test/lib/utils/log-shim.js
 delete mode 100644 deps/npm/test/lib/utils/proc-log-listener.js
 delete mode 100644 deps/npm/test/lib/utils/setup-log.js
 create mode 100644 deps/npm/test/lib/utils/timers.js
   ✔  Patches applied
--------------------------------------------------------------------------------
--------------------------------- New Message ----------------------------------
deps: upgrade npm to 8.2.0

PR-URL: #41065

Fixes: https://github.com/add

Fixes: https://github.com/always

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

Reviewed-By: Luigi Pinca luigipinca@gmail.com

Reviewed-By: Rich Trott rtrott@gmail.com


[master cdce283f1a] deps: upgrade npm to 8.2.0

Author: npm team ops+robot@npmjs.com

Date: Thu Dec 2 22:04:46 2021 +0000

287 files changed, 5497 insertions(+), 5996 deletions(-)

create mode 100644 deps/npm/docs/content/using-npm/logging.md

create mode 100644 deps/npm/docs/output/using-npm/logging.html

delete mode 100644 deps/npm/lib/utils/cleanup-log-files.js

create mode 100644 deps/npm/lib/utils/display.js

delete mode 100644 deps/npm/lib/utils/get-project-scope.js

create mode 100644 deps/npm/lib/utils/log-file.js

create mode 100644 deps/npm/lib/utils/log-shim.js

delete mode 100644 deps/npm/lib/utils/proc-log-listener.js

delete mode 100644 deps/npm/lib/utils/setup-log.js

create mode 100644 deps/npm/lib/utils/timers.js

create mode 100644 deps/npm/lib/utils/with-chown-sync.js

create mode 100644 deps/npm/man/man7/logging.7

delete mode 100644 deps/npm/node_modules/code-point-at/index.js

delete mode 100644 deps/npm/node_modules/code-point-at/license

delete mode 100644 deps/npm/node_modules/code-point-at/package.json

delete mode 100644 deps/npm/node_modules/code-point-at/readme.md

delete mode 100644 deps/npm/node_modules/node-gyp/node_modules/aproba/LICENSE

delete mode 100644 deps/npm/node_modules/node-gyp/node_modules/aproba/index.js

delete mode 100644 deps/npm/node_modules/node-gyp/node_modules/aproba/package.json

delete mode 100644 deps/npm/node_modules/node-gyp/node_modules/gauge/LICENSE

delete mode 100644 deps/npm/node_modules/node-gyp/node_modules/gauge/base-theme.js

delete mode 100644 deps/npm/node_modules/node-gyp/node_modules/gauge/error.js

delete mode 100644 deps/npm/node_modules/node-gyp/node_modules/gauge/has-color.js

delete mode 100644 deps/npm/node_modules/node-gyp/node_modules/gauge/index.js

delete mode 100644 deps/npm/node_modules/node-gyp/node_modules/gauge/package.json

delete mode 100644 deps/npm/node_modules/node-gyp/node_modules/gauge/plumbing.js

delete mode 100644 deps/npm/node_modules/node-gyp/node_modules/gauge/process.js

delete mode 100644 deps/npm/node_modules/node-gyp/node_modules/gauge/progress-bar.js

delete mode 100644 deps/npm/node_modules/node-gyp/node_modules/gauge/render-template.js

delete mode 100644 deps/npm/node_modules/node-gyp/node_modules/gauge/set-immediate.js

delete mode 100644 deps/npm/node_modules/node-gyp/node_modules/gauge/set-interval.js

delete mode 100644 deps/npm/node_modules/node-gyp/node_modules/gauge/spin.js

delete mode 100644 deps/npm/node_modules/node-gyp/node_modules/gauge/template-item.js

delete mode 100644 deps/npm/node_modules/node-gyp/node_modules/gauge/theme-set.js

delete mode 100644 deps/npm/node_modules/node-gyp/node_modules/gauge/themes.js

delete mode 100644 deps/npm/node_modules/node-gyp/node_modules/gauge/wide-truncate.js

delete mode 100644 deps/npm/node_modules/node-gyp/node_modules/is-fullwidth-code-point/index.js

delete mode 100644 deps/npm/node_modules/node-gyp/node_modules/is-fullwidth-code-point/license

delete mode 100644 deps/npm/node_modules/node-gyp/node_modules/is-fullwidth-code-point/package.json

delete mode 100644 deps/npm/node_modules/node-gyp/node_modules/npmlog/LICENSE

delete mode 100644 deps/npm/node_modules/node-gyp/node_modules/npmlog/log.js

delete mode 100644 deps/npm/node_modules/node-gyp/node_modules/npmlog/package.json

delete mode 100644 deps/npm/node_modules/node-gyp/node_modules/string-width/index.js

delete mode 100644 deps/npm/node_modules/node-gyp/node_modules/string-width/license

delete mode 100644 deps/npm/node_modules/node-gyp/node_modules/string-width/package.json

delete mode 100644 deps/npm/node_modules/npmlog/node_modules/are-we-there-yet/LICENSE.md

delete mode 100644 deps/npm/node_modules/npmlog/node_modules/are-we-there-yet/lib/index.js

delete mode 100644 deps/npm/node_modules/npmlog/node_modules/are-we-there-yet/lib/tracker-base.js

delete mode 100644 deps/npm/node_modules/npmlog/node_modules/are-we-there-yet/lib/tracker-group.js

delete mode 100644 deps/npm/node_modules/npmlog/node_modules/are-we-there-yet/lib/tracker-stream.js

delete mode 100644 deps/npm/node_modules/npmlog/node_modules/are-we-there-yet/lib/tracker.js

delete mode 100644 deps/npm/node_modules/npmlog/node_modules/are-we-there-yet/package.json

delete mode 100644 deps/npm/node_modules/number-is-nan/index.js

delete mode 100644 deps/npm/node_modules/number-is-nan/license

delete mode 100644 deps/npm/node_modules/number-is-nan/package.json

delete mode 100644 deps/npm/node_modules/number-is-nan/readme.md

create mode 100644 deps/npm/tap-snapshots/test/lib/utils/log-file.js.test.cjs

create mode 100644 deps/npm/test/fixtures/clean-snapshot.js

create mode 100644 deps/npm/test/fixtures/mock-globals.js

create mode 100644 deps/npm/test/fixtures/mock-logs.js

rewrite deps/npm/test/fixtures/mock-npm.js (65%)

rewrite deps/npm/test/lib/cli.js (63%)

rewrite deps/npm/test/lib/commands/dedupe.js (69%)

rewrite deps/npm/test/lib/commands/find-dupes.js (80%)

rewrite deps/npm/test/lib/commands/prune.js (75%)

rewrite deps/npm/test/lib/commands/version.js (82%)

create mode 100644 deps/npm/test/lib/fixtures/mock-globals.js

rewrite deps/npm/test/lib/load-all.js (70%)

delete mode 100644 deps/npm/test/lib/utils/cleanup-log-files.js

create mode 100644 deps/npm/test/lib/utils/display.js

rewrite deps/npm/test/lib/utils/exit-handler.js (81%)

delete mode 100644 deps/npm/test/lib/utils/get-project-scope.js

rewrite deps/npm/test/lib/utils/is-windows-bash.js (65%)

create mode 100644 deps/npm/test/lib/utils/log-file.js

create mode 100644 deps/npm/test/lib/utils/log-shim.js

delete mode 100644 deps/npm/test/lib/utils/proc-log-listener.js

delete mode 100644 deps/npm/test/lib/utils/setup-log.js

create mode 100644 deps/npm/test/lib/utils/timers.js

✖  cdce283f1a93b82c211466f899c6ca812ce6fb51

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

✖  3: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/1541037621

Trott pushed a commit that referenced this pull request Dec 5, 2021
@npm-robot @Trott
deps: upgrade npm to 8.2.0
b323cec 
PR-URL: #41065
Reviewed-By: Myles Borins <myles.borins@gmail.com>
Reviewed-By: Luigi Pinca <luigipinca@gmail.com>
Reviewed-By: Rich Trott <rtrott@gmail.com>
@Trott
Copy link
Member

Trott commented Dec 5, 2021

Landed in b323cec

@Trott Trott closed this Dec 5, 2021
This was referenced Dec 6, 2021
Open
Open
danielleadams pushed a commit that referenced this pull request Dec 13, 2021
@npm-robot @danielleadams
deps: upgrade npm to 8.2.0
4a7bf51 
PR-URL: #41065
Reviewed-By: Myles Borins <myles.borins@gmail.com>
Reviewed-By: Luigi Pinca <luigipinca@gmail.com>
Reviewed-By: Rich Trott <rtrott@gmail.com>
danielleadams pushed a commit that referenced this pull request Dec 14, 2021
@npm-robot @danielleadams
deps: upgrade npm to 8.2.0
3a1d952 
PR-URL: #41065
Reviewed-By: Myles Borins <myles.borins@gmail.com>
Reviewed-By: Luigi Pinca <luigipinca@gmail.com>
Reviewed-By: Rich Trott <rtrott@gmail.com>
@danielleadams danielleadams mentioned this pull request Dec 14, 2021
Merged
danielleadams pushed a commit that referenced this pull request Jan 31, 2022
@npm-robot @danielleadams
deps: upgrade npm to 8.2.0
4d16c69 
PR-URL: #41065
Reviewed-By: Myles Borins <myles.borins@gmail.com>
Reviewed-By: Luigi Pinca <luigipinca@gmail.com>
Reviewed-By: Rich Trott <rtrott@gmail.com>
danielleadams pushed a commit that referenced this pull request Jan 31, 2022
@npm-robot @danielleadams
deps: upgrade npm to 8.2.0
748d9de 
PR-URL: #41065
Reviewed-By: Myles Borins <myles.borins@gmail.com>
Reviewed-By: Luigi Pinca <luigipinca@gmail.com>
Reviewed-By: Rich Trott <rtrott@gmail.com>
Linkgoron pushed a commit to Linkgoron/node that referenced this pull request Jan 31, 2022
@npm-robot @Linkgoron
deps: upgrade npm to 8.2.0
fca93f4 
PR-URL: nodejs#41065
Reviewed-By: Myles Borins <myles.borins@gmail.com>
Reviewed-By: Luigi Pinca <luigipinca@gmail.com>
Reviewed-By: Rich Trott <rtrott@gmail.com>
danielleadams pushed a commit that referenced this pull request Feb 1, 2022
@npm-robot @danielleadams
deps: upgrade npm to 8.2.0
3c13bbd 
PR-URL: #41065
Reviewed-By: Myles Borins <myles.borins@gmail.com>
Reviewed-By: Luigi Pinca <luigipinca@gmail.com>
Reviewed-By: Rich Trott <rtrott@gmail.com>
@danielleadams danielleadams mentioned this pull request Feb 1, 2022
Merged
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

@lpinca lpinca lpinca 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.

None yet
7 participants
@npm-robot @wraithgar @nodejs-github-bot @Trott @MylesBorins @lpinca @targos