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

doc: correct attribution in v20.6.0 changelog #50564

Merged
merged 1 commit into from Nov 7, 2023

Conversation

JakobJingleheimer
Copy link
Contributor

@JakobJingleheimer JakobJingleheimer 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-bot nodejs-github-bot added the release Issues and PRs related to Node.js releases. label Nov 5, 2023
@JakobJingleheimer JakobJingleheimer added the fast-track PRs that do not need to wait for 48 hours to land. label Nov 5, 2023
Copy link
Contributor

github-actions bot commented Nov 5, 2023

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

Copy link
Member

@UlisesGascon UlisesGascon 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
Contributor Author

Cheers!

Could you 👍 the fast-track 🙂

@aduh95 aduh95 added the author ready PRs that have at least one approval, no pending requests for changes, and a CI started. label Nov 6, 2023
@JakobJingleheimer JakobJingleheimer 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-bot nodejs-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

@JakobJingleheimer JakobJingleheimer 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-bot nodejs-github-bot removed the commit-queue Add this label to land a pull request using GitHub Actions. label Nov 7, 2023
@nodejs-github-bot nodejs-github-bot merged commit c5e1fd0 into main Nov 7, 2023
35 checks passed
@nodejs-github-bot nodejs-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 <duhamelantoine1995@gmail.com>
Reviewed-By: Claudio Wunder <cwunder@gnome.org>
targos pushed a commit that referenced this pull request Nov 11, 2023
PR-URL: #50564
Reviewed-By: Antoine du Hamel <duhamelantoine1995@gmail.com>
Reviewed-By: Claudio Wunder <cwunder@gnome.org>
targos pushed a commit that referenced this pull request Nov 14, 2023
PR-URL: #50564
Reviewed-By: Antoine du Hamel <duhamelantoine1995@gmail.com>
Reviewed-By: Claudio Wunder <cwunder@gnome.org>
UlisesGascon pushed a commit that referenced this pull request Dec 11, 2023
PR-URL: #50564
Reviewed-By: Antoine du Hamel <duhamelantoine1995@gmail.com>
Reviewed-By: Claudio Wunder <cwunder@gnome.org>
@UlisesGascon UlisesGascon mentioned this pull request Dec 12, 2023
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. commit-queue-squash Add this label to instruct the Commit Queue to squash all the PR commits into the first one. fast-track PRs that do not need to wait for 48 hours to land. release Issues and PRs related to Node.js releases.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

5 participants