Skip to content

deps: upgrade npm to 7.5.3 - #37283

Closed
ruyadorno wants to merge 1 commit into
nodejs:masterfrom
npm:npm-7.5.3
Closed

deps: upgrade npm to 7.5.3#37283
ruyadorno wants to merge 1 commit into
nodejs:masterfrom
npm:npm-7.5.3

Conversation

@ruyadorno

@ruyadornoruyadorno commented Feb 8, 2021

Copy link
Copy Markdown
Member

v7.5.3 (2021-02-08)

BUG FIXES

DEPENDENCIES

  • 3294fed6fpacote@11.2.5
    • prevent infinite recursion in git dep preparation
  • 0f7a3a87cread-package-json-fast@2.0.1
    • avoid duplicating optionalDependencies as dependencies in package.json
  • 6f46b0f7finit-package-json@2.0.2
  • df4f65acc@npmcli/arborist@2.2.0
  • 7038c2ff4@npmcli/run-script@1.8.2
  • 54cd4c87alibnpmversion@1.0.8
  • 9ab36aae4graceful-fs@4.2.5
  • e1822cf27@npmcli/installed-package-contents@1.0.7

@nodejs-github-botnodejs-github-bot added the npm Issues and PRs related to the npm client dependency or the npm registry. label Feb 8, 2021
@nodejs-github-bot

nodejs-github-bot commented Feb 8, 2021

Copy link
Copy Markdown
Collaborator

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

Trott
Trott approved these changes Feb 8, 2021
View reviewed changes

@TrottTrott left a comment

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

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

Rubber stamp LGTM

MylesBorins
MylesBorins approved these changes Feb 9, 2021
View reviewed changes

@MylesBorinsMylesBorins left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

LGTM

@MylesBorins

MylesBorins commented Feb 9, 2021

Copy link
Copy Markdown
Contributor

fast track?

@MylesBorinsMylesBorins added the fast-track PRs that do not need to wait for 48 hours to land. label Feb 9, 2021
@nodejs-github-bot

nodejs-github-bot commented Feb 9, 2021

Copy link
Copy Markdown
Collaborator

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

@nodejs-github-bot

nodejs-github-bot commented Feb 9, 2021

Copy link
Copy Markdown
Collaborator

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

MylesBorins pushed a commit that referenced this pull request Feb 9, 2021
@ruyadorno@MylesBorins
deps: upgrade npm to 7.5.3
730ba5c
PR-URL: #37283
Reviewed-By: Rich Trott <rtrott@gmail.com>
Reviewed-By: Myles Borins <myles.borins@gmail.com>
Reviewed-By: Luigi Pinca <luigipinca@gmail.com>
@MylesBorins

MylesBorins commented Feb 9, 2021

Copy link
Copy Markdown
Contributor

landed in 730ba5c

danielleadams pushed a commit that referenced this pull request Feb 16, 2021
@ruyadorno@danielleadams
deps: upgrade npm to 7.5.3
d1c1724
PR-URL: #37283
Reviewed-By: Rich Trott <rtrott@gmail.com>
Reviewed-By: Myles Borins <myles.borins@gmail.com>
Reviewed-By: Luigi Pinca <luigipinca@gmail.com>
This was referenced Feb 16, 2021
Closed
Merged
@wraithgar
wraithgar deleted the npm-7.5.3 branch June 26, 2025 15:42
Sign up for freeto join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

@TrottTrottTrott approved these changes
@lpincalpincalpinca approved these changes
@marsonyamarsonyamarsonya approved these changes
+1 more reviewer
@MylesBorinsMylesBorinsMylesBorins approved these changes
Reviewers whose approvals may not affect merge requirements

Assignees

No one assigned

Labels

fast-trackPRs that do not need to wait for 48 hours to land.npmIssues 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.

7 participants

@ruyadorno@nodejs-github-bot@MylesBorins@Trott@lpinca@marsonya@targos