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

deps: upgrade npm to 10.5.1 by npm-cli-bot · Pull Request #52351 · nodejs/node · GitHub

/ node Public

deps: upgrade npm to 10.5.1 - #52351

Merged
nodejs-github-bot merged 1 commit into
nodejs:mainfrom
npm:npm-v10.5.1-main
Apr 7, 2024
ConversationCommits (1)ChecksFiles changed
Merged

deps: upgrade npm to 10.5.1#52351
nodejs-github-bot merged 1 commit into
nodejs:mainfrom
npm:npm-v10.5.1-main

Conversation

npm-cli-bot commented Apr 3, 2024

Copy link
Copy Markdown
Contributor

10.5.1 (2024-04-03)

Bug Fixes

Documentation

Dependencies

Chores

nodejs-github-bot commented Apr 3, 2024

Copy link
Copy Markdown
Collaborator

Review requested:

  • @nodejs/security-wg

nodejs-github-bot added 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 Apr 3, 2024

github-actions Bot commented Apr 3, 2024

Copy link
Copy Markdown
Contributor

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

lukekarrys added the request-ci Add this label to start a Jenkins CI on a PR. label Apr 3, 2024
mhdawson approved these changes Apr 3, 2024
View reviewed changes

mhdawson 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.

Choose a reason Spam Abuse Off Topic Outdated Duplicate Resolved Low Quality

Rubber stamp LGTM

dominicfraser mentioned this pull request Apr 3, 2024
Closed
2 tasks
github-actions Bot removed the request-ci Add this label to start a Jenkins CI on a PR. label Apr 3, 2024

nodejs-github-bot commented Apr 3, 2024

Copy link
Copy Markdown
Collaborator

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

VoltrexKeyva approved these changes Apr 3, 2024
View reviewed changes

VoltrexKeyva 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.

Choose a reason Spam Abuse Off Topic Outdated Duplicate Resolved Low Quality

RSLGTM

nodejs-github-bot commented Apr 3, 2024

Copy link
Copy Markdown
Collaborator

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

nodejs-github-bot commented Apr 3, 2024

Copy link
Copy Markdown
Collaborator

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

nodejs-github-bot commented Apr 4, 2024

Copy link
Copy Markdown
Collaborator

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

UlisesGascon approved these changes Apr 4, 2024
View reviewed changes

UlisesGascon 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.

Choose a reason Spam Abuse Off Topic Outdated Duplicate Resolved Low Quality

LGTM! I checked the licenses too following this script

wraithgar commented Apr 4, 2024

Copy link
Copy Markdown
Contributor

LGTM! I checked the licenses too following this script

npm also does license checking as part of its release process. The hiccup in the past was because blueOak wasn't explicitly in the openjs list. That is the only non CC-BY-3.0 license in our license linter, so unless you see that file change this shouldn't be worth thinking about

Current .licensee.json file:

{
  "licenses": {
    "spdx": [
      "CC-BY-3.0"
    ],
    "blueOak": "bronze"
  },
  "corrections": true
}

In the npm repo you can run npm run licenses to validate against this list.

nodejs-github-bot commented Apr 4, 2024

Copy link
Copy Markdown
Collaborator

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

melroy89 mentioned this pull request Apr 5, 2024
Closed
2 tasks

nodejs-github-bot commented Apr 6, 2024

Copy link
Copy Markdown
Collaborator

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

richardlau added the commit-queue Add this label to land a pull request using GitHub Actions. label Apr 7, 2024
nodejs-github-bot removed the commit-queue Add this label to land a pull request using GitHub Actions. label Apr 7, 2024
nodejs-github-bot merged commit ad86a12 into nodejs:main Apr 7, 2024

nodejs-github-bot commented Apr 7, 2024

Copy link
Copy Markdown
Collaborator

Landed in ad86a12

RafaelGSS mentioned this pull request Apr 15, 2024
Merged
richardlau pushed a commit that referenced this pull request Apr 26, 2024
deps: upgrade npm to 10.5.1
052b0ba 
PR-URL: #52351
Reviewed-By: Luke Karrys <luke@lukekarrys.com>
Reviewed-By: Michael Dawson <midawson@redhat.com>
Reviewed-By: Mohammed Keyvanzadeh <mohammadkeyvanzade94@gmail.com>
Reviewed-By: Marco Ippolito <marcoippolito54@gmail.com>
Reviewed-By: Ulises Gascón <ulisesgascongonzalez@gmail.com>
Reviewed-By: Luigi Pinca <luigipinca@gmail.com>
marco-ippolito pushed a commit that referenced this pull request May 2, 2024
deps: upgrade npm to 10.5.1
24bd695 
PR-URL: #52351
Reviewed-By: Luke Karrys <luke@lukekarrys.com>
Reviewed-By: Michael Dawson <midawson@redhat.com>
Reviewed-By: Mohammed Keyvanzadeh <mohammadkeyvanzade94@gmail.com>
Reviewed-By: Marco Ippolito <marcoippolito54@gmail.com>
Reviewed-By: Ulises Gascón <ulisesgascongonzalez@gmail.com>
Reviewed-By: Luigi Pinca <luigipinca@gmail.com>
marco-ippolito mentioned this pull request May 2, 2024
Merged
marco-ippolito pushed a commit that referenced this pull request May 3, 2024
deps: upgrade npm to 10.5.1
0a103e9 
PR-URL: #52351
Reviewed-By: Luke Karrys <luke@lukekarrys.com>
Reviewed-By: Michael Dawson <midawson@redhat.com>
Reviewed-By: Mohammed Keyvanzadeh <mohammadkeyvanzade94@gmail.com>
Reviewed-By: Marco Ippolito <marcoippolito54@gmail.com>
Reviewed-By: Ulises Gascón <ulisesgascongonzalez@gmail.com>
Reviewed-By: Luigi Pinca <luigipinca@gmail.com>
richardlau mentioned this pull request May 16, 2024
Merged
wraithgar deleted the npm-v10.5.1-main branch June 26, 2025 15:46
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
Show hidden characters
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

lpinca lpinca approved these changes
UlisesGascon UlisesGascon approved these changes
marco-ippolito marco-ippolito approved these changes
+3 more reviewers
lukekarrys lukekarrys approved these changes
mhdawson mhdawson approved these changes
VoltrexKeyva VoltrexKeyva approved these changes
Reviewers whose approvals may not affect merge requirements

Assignees

No one assigned

Labels

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.

10 participants

Back | FazBrowse Home | New Git URL