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

[v14.x backport] deps: update corepack to 0.14.1 #43760

Closed
wants to merge 8 commits into from

Conversation

@nodejs-github-bot nodejs-github-bot added dependencies Pull requests that update a dependency file. needs-ci PRs that need a full CI run. v14.x labels Jul 10, 2022
arcanis and others added 2 commits July 19, 2022 19:44
PR-URL: nodejs#43402
Reviewed-By: Antoine du Hamel <[email protected]>
Reviewed-By: Mohammed Keyvanzadeh <[email protected]>
Reviewed-By: Benjamin Gruenbaum <[email protected]>
Reviewed-By: James M Snell <[email protected]>
Reviewed-By: Darshan Sen <[email protected]>
Reviewed-By: Jiawen Geng <[email protected]>
PR-URL: nodejs#43748
Reviewed-By: Antoine du Hamel <[email protected]>
Reviewed-By: Tobias Nießen <[email protected]>
Reviewed-By: Mohammed Keyvanzadeh <[email protected]>
@aduh95 aduh95 force-pushed the backport-corepack-updates branch from dec6cda to 21e45c8 Compare July 19, 2022 17:49
PR-URL: nodejs#43965
Reviewed-By: Mohammed Keyvanzadeh <[email protected]>
Reviewed-By: Antoine du Hamel <[email protected]>
Reviewed-By: Mestery <[email protected]>
@aduh95 aduh95 changed the title [v14.x backport] deps: update corepack to 0.12.0 [v14.x backport] deps: update corepack to 0.12.1 Jul 24, 2022
Copy link
Member

@F3n67u F3n67u left a comment

Choose a reason for hiding this comment

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

RSLGTM

@F3n67u F3n67u added author ready PRs that have at least one approval, no pending requests for changes, and a CI started. request-ci Add this label to start a Jenkins CI on a PR. commit-queue Add this label to land a pull request using GitHub Actions. labels Jul 25, 2022
@F3n67u F3n67u added the commit-queue-rebase Add this label to allow the Commit Queue to land a PR in several commits. label Jul 25, 2022
@richardlau
Copy link
Member

FWIW I don't believe the commit queue works for any branch other than main. For one thing when https://github.com/nodejs/node/blob/main/.github/workflows/commit-queue.yml is run on the cron timer I believe github.ref_name is always refs/heads/main so the PR list is always filtered to PR's with a base branch of main. A second blocker would be that the github bot user isn't a backporter so can't land things on the staging branch.

@github-actions github-actions bot removed the request-ci Add this label to start a Jenkins CI on a PR. label Jul 25, 2022
@nodejs-github-bot
Copy link
Collaborator

@F3n67u F3n67u removed commit-queue Add this label to land a pull request using GitHub Actions. commit-queue-rebase Add this label to allow the Commit Queue to land a PR in several commits. labels Jul 25, 2022
@F3n67u
Copy link
Member

F3n67u commented Jul 25, 2022

FWIW I don't believe the commit queue works for any branch other than main. For one thing when https://github.com/nodejs/node/blob/main/.github/workflows/commit-queue.yml is run on the cron timer I believe github.ref_name is always refs/heads/main so the PR list is always filtered to PR's with a base branch of main. A second blocker would be that the github bot user isn't a backporter so can't land things on the staging branch.

Got it. Removed the commit-queue label.

PR-URL: nodejs#44159
Reviewed-By: Mohammed Keyvanzadeh <[email protected]>
Reviewed-By: Antoine du Hamel <[email protected]>
Reviewed-By: Feng Yu <[email protected]>
@aduh95 aduh95 changed the title [v14.x backport] deps: update corepack to 0.12.1 [v14.x backport] deps: update corepack to 0.12.3 Aug 15, 2022
PR-URL: nodejs#44229
Reviewed-By: Antoine du Hamel <[email protected]>
Reviewed-By: Mohammed Keyvanzadeh <[email protected]>
Reviewed-By: Feng Yu <[email protected]>
Reviewed-By: Rich Trott <[email protected]>
PR-URL: nodejs#44318
Reviewed-By: Tobias Nießen <[email protected]>
Reviewed-By: Rich Trott <[email protected]>
Reviewed-By: Antoine du Hamel <[email protected]>
@aduh95 aduh95 changed the title [v14.x backport] deps: update corepack to 0.12.3 [v14.x backport] deps: update corepack to 0.13.0 Aug 21, 2022
@aduh95 aduh95 changed the title [v14.x backport] deps: update corepack to 0.13.0 [v14.x backport] deps: update corepack to 0.14.0 Sep 5, 2022
PR-URL: nodejs#44509
Reviewed-By: Mohammed Keyvanzadeh <[email protected]>
Reviewed-By: Darshan Sen <[email protected]>
Reviewed-By: Tobias Nießen <[email protected]>
Reviewed-By: Feng Yu <[email protected]>
Reviewed-By: Antoine du Hamel <[email protected]>
@aduh95 aduh95 changed the title [v14.x backport] deps: update corepack to 0.14.0 [v14.x backport] deps: update corepack to 0.14.1 Sep 18, 2022
PR-URL: nodejs#44704
Reviewed-By: Moshe Atlow <[email protected]>
Reviewed-By: Antoine du Hamel <[email protected]>
danielleadams pushed a commit that referenced this pull request Oct 1, 2022
PR-URL: #43402
Backport-PR-URL: #43760
Reviewed-By: Antoine du Hamel <[email protected]>
Reviewed-By: Mohammed Keyvanzadeh <[email protected]>
Reviewed-By: Benjamin Gruenbaum <[email protected]>
Reviewed-By: James M Snell <[email protected]>
Reviewed-By: Darshan Sen <[email protected]>
Reviewed-By: Jiawen Geng <[email protected]>
danielleadams pushed a commit that referenced this pull request Oct 1, 2022
PR-URL: #43748
Backport-PR-URL: #43760
Reviewed-By: Antoine du Hamel <[email protected]>
Reviewed-By: Tobias Nießen <[email protected]>
Reviewed-By: Mohammed Keyvanzadeh <[email protected]>
danielleadams pushed a commit that referenced this pull request Oct 1, 2022
PR-URL: #43965
Backport-PR-URL: #43760
Reviewed-By: Mohammed Keyvanzadeh <[email protected]>
Reviewed-By: Antoine du Hamel <[email protected]>
Reviewed-By: Mestery <[email protected]>
danielleadams pushed a commit that referenced this pull request Oct 1, 2022
PR-URL: #44159
Backport-PR-URL: #43760
Reviewed-By: Mohammed Keyvanzadeh <[email protected]>
Reviewed-By: Antoine du Hamel <[email protected]>
Reviewed-By: Feng Yu <[email protected]>
danielleadams pushed a commit that referenced this pull request Oct 1, 2022
PR-URL: #44229
Backport-PR-URL: #43760
Reviewed-By: Antoine du Hamel <[email protected]>
Reviewed-By: Mohammed Keyvanzadeh <[email protected]>
Reviewed-By: Feng Yu <[email protected]>
Reviewed-By: Rich Trott <[email protected]>
danielleadams pushed a commit that referenced this pull request Oct 1, 2022
PR-URL: #44318
Backport-PR-URL: #43760
Reviewed-By: Tobias Nießen <[email protected]>
Reviewed-By: Rich Trott <[email protected]>
Reviewed-By: Antoine du Hamel <[email protected]>
danielleadams pushed a commit that referenced this pull request Oct 1, 2022
PR-URL: #44509
Backport-PR-URL: #43760
Reviewed-By: Mohammed Keyvanzadeh <[email protected]>
Reviewed-By: Darshan Sen <[email protected]>
Reviewed-By: Tobias Nießen <[email protected]>
Reviewed-By: Feng Yu <[email protected]>
Reviewed-By: Antoine du Hamel <[email protected]>
danielleadams pushed a commit that referenced this pull request Oct 1, 2022
PR-URL: #44704
Backport-PR-URL: #43760
Reviewed-By: Moshe Atlow <[email protected]>
Reviewed-By: Antoine du Hamel <[email protected]>
@danielleadams
Copy link
Contributor

Landed in 4f83d75...a1f8e4d

@aduh95 aduh95 deleted the backport-corepack-updates branch October 8, 2022 16:15
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
author ready PRs that have at least one approval, no pending requests for changes, and a CI started. dependencies Pull requests that update a dependency file. needs-ci PRs that need a full CI run.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

9 participants