Skip to content

Conversation

ruyadorno
Copy link
Member

v7.6.3 (2021-03-11)

DOCUMENTATION

DEPENDENCIES

v7.6.2 (2021-03-09)

BUG FIXES

DEPENDENCIES

@nodejs-github-bot nodejs-github-bot added needs-ci PRs that need a full CI run. npm Issues and PRs related to the npm client dependency or the npm registry. labels Mar 11, 2021
@nodejs-github-bot
Copy link
Collaborator

@ruyadorno ruyadorno added the fast-track PRs that do not need to wait for 48 hours to land. label Mar 11, 2021
@ruyadorno
Copy link
Member Author

fast-track?

@nodejs-github-bot
Copy link
Collaborator

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

@nodejs-github-bot

This comment has been minimized.

@aduh95
Copy link
Contributor

aduh95 commented Mar 12, 2021

Do we really need a passing Jenkins CI for an npm update? I believe we don't test anything related to npm that the GitHub Actions CI doesn't already check.

@targos
Copy link
Member

targos commented Mar 12, 2021

@ruyadorno
Copy link
Member Author

@targos I don't think that one is actually running on CI 😅 I have seen it broken on the main branch and had to manually fix it later many times

@nodejs-github-bot

This comment has been minimized.

@nodejs-github-bot

This comment has been minimized.

@nodejs-github-bot
Copy link
Collaborator

@nodejs-github-bot
Copy link
Collaborator

Trott pushed a commit that referenced this pull request Mar 13, 2021
PR-URL: #37721
Reviewed-By: Rich Trott <[email protected]>
Reviewed-By: Luigi Pinca <[email protected]>
Reviewed-By: Antoine du Hamel <[email protected]>
@Trott
Copy link
Member

Trott commented Mar 13, 2021

Landed in 5fdcfb3

@Trott Trott closed this Mar 13, 2021
@Trott
Copy link
Member

Trott commented Mar 13, 2021

@aduh95 we have at least one npm test: https://github.com/nodejs/node/blob/master/test/parallel/test-npm-install.js

There's also test-release-npm.js and test-npm-version.js.

danielleadams pushed a commit that referenced this pull request Mar 16, 2021
PR-URL: #37721
Reviewed-By: Rich Trott <[email protected]>
Reviewed-By: Luigi Pinca <[email protected]>
Reviewed-By: Antoine du Hamel <[email protected]>
danielleadams pushed a commit that referenced this pull request Mar 16, 2021
PR-URL: #37721
Reviewed-By: Rich Trott <[email protected]>
Reviewed-By: Luigi Pinca <[email protected]>
Reviewed-By: Antoine du Hamel <[email protected]>
@wraithgar wraithgar deleted the npm-7.6.3 branch June 26, 2025 15:43
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
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
Development

Successfully merging this pull request may close these issues.

6 participants