Skip to content

Conversation

@JakobJingleheimer
Copy link
Member

@JakobJingleheimerJakobJingleheimer commented Nov 5, 2023

Not sure what happened when these were generated, but the original PR that introduced the feature was omitted and co-authorship on both PRs was lost.

Related: nodejs/nodejs.org#6095

@nodejs-github-botnodejs-github-bot added the release Issues and PRs related to Node.js releases. label Nov 5, 2023
@JakobJingleheimerJakobJingleheimer added the fast-track PRs that do not need to wait for 48 hours to land. label Nov 5, 2023
@github-actions
Copy link
Contributor

Fast-track has been requested by @JakobJingleheimer. Please 👍 to approve.

Copy link
Member

@UlisesGasconUlisesGascon left a comment

Choose a reason for hiding this comment

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

Not sure why/how I introduced this bug, but thanks for the fix 🙂

@JakobJingleheimer
Copy link
MemberAuthor

Cheers!

Could you 👍 the fast-track 🙂

@aduh95aduh95 added the author ready PRs that have at least one approval, no pending requests for changes, and a CI started. label Nov 6, 2023
@JakobJingleheimerJakobJingleheimer added commit-queue Add this label to land a pull request using GitHub Actions. commit-queue-squash Add this label to instruct the Commit Queue to squash all the PR commits into the first one. labels Nov 6, 2023
@nodejs-github-botnodejs-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 Nov 6, 2023
@nodejs-github-bot
Copy link
Collaborator

Commit Queue failed
- Loading data for nodejs/node/pull/50564 ✔ Done loading data for nodejs/node/pull/50564 ----------------------------------- PR info ------------------------------------ Title doc: correct attribution in v20.6.0 changelog (#50564) ⚠ Could not retrieve the email or name of the PR author's from user's GitHub profile! Branch JakobJingleheimer:correct-20.6.0-changelog -> nodejs:main Labels fast-track, author ready, release, commit-queue-squash Commits 1 - doc: correct attribution in v20.6.0 changelog Committers 1 - GitHub PR-URL: https://github.com/nodejs/node/pull/50564 Reviewed-By: Antoine du Hamel ------------------------------ Generated metadata ------------------------------ PR-URL: https://github.com/nodejs/node/pull/50564 Reviewed-By: Antoine du Hamel -------------------------------------------------------------------------------- ℹ This PR was created on Sun, 05 Nov 2023 15:23:59 GMT ✔ Approvals: 1 ✔ - Antoine du Hamel (@aduh95) (TSC): https://github.com/nodejs/node/pull/50564#pullrequestreview-1716407245 ℹ This PR is being fast-tracked ✘ This PR needs to wait 136 more hours to land (or 16 hours if there is one more approval) (or 0 hours if there is 1 more approval (👍) of the fast-track request from collaborators). ✔ Last GitHub CI successful ℹ Green GitHub CI is sufficient -------------------------------------------------------------------------------- ✔ Aborted `git node land` session in /home/runner/work/node/node/.ncu
https://github.com/nodejs/node/actions/runs/6777728206

@JakobJingleheimerJakobJingleheimer added commit-queue Add this label to land a pull request using GitHub Actions. and removed commit-queue-failed An error occurred while landing this pull request using GitHub Actions. labels Nov 7, 2023
@nodejs-github-botnodejs-github-bot removed the commit-queue Add this label to land a pull request using GitHub Actions. label Nov 7, 2023
@nodejs-github-botnodejs-github-bot merged commit c5e1fd0 into mainNov 7, 2023
@nodejs-github-botnodejs-github-bot deleted the correct-20.6.0-changelog branch November 7, 2023 10:25
@nodejs-github-bot
Copy link
Collaborator

Landed in c5e1fd0

anonrig pushed a commit to anonrig/node that referenced this pull request Nov 9, 2023
PR-URL: nodejs#50564 Reviewed-By: Antoine du Hamel <[email protected]> Reviewed-By: Claudio Wunder <[email protected]>
targos pushed a commit that referenced this pull request Nov 11, 2023
PR-URL: #50564 Reviewed-By: Antoine du Hamel <[email protected]> Reviewed-By: Claudio Wunder <[email protected]>
@targostargos mentioned this pull request Nov 12, 2023
targos pushed a commit that referenced this pull request Nov 14, 2023
PR-URL: #50564 Reviewed-By: Antoine du Hamel <[email protected]> Reviewed-By: Claudio Wunder <[email protected]>
UlisesGascon pushed a commit that referenced this pull request Dec 11, 2023
PR-URL: #50564 Reviewed-By: Antoine du Hamel <[email protected]> Reviewed-By: Claudio Wunder <[email protected]>
@UlisesGasconUlisesGascon mentioned this pull request Dec 12, 2023
@targostargos removed the release Issues and PRs related to Node.js releases. label May 13, 2024
Sign up for freeto join this conversation on GitHub. Already have an account? Sign in to comment

Labels

author readyPRs that have at least one approval, no pending requests for changes, and a CI started.commit-queue-squashAdd this label to instruct the Commit Queue to squash all the PR commits into the first one.fast-trackPRs that do not need to wait for 48 hours to land.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

7 participants

@JakobJingleheimer@nodejs-github-bot@UlisesGascon@ovflowd@aduh95@targos