Skip to content

Conversation

rluvaton
Copy link
Member

the email should be wrapped with <<{email}>> and not <{email}>

const [, name, email] = /^ {2}\*\*([^*]+)\*\* <<(.+)>>/.exec(line);

@rluvaton rluvaton added the fast-track PRs that do not need to wait for 48 hours to land. label Mar 30, 2024
@rluvaton rluvaton requested a review from zcbenz March 30, 2024 21:33
Copy link
Contributor

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

@nodejs-github-bot nodejs-github-bot added the doc Issues and PRs related to the documentations. label Mar 30, 2024
Copy link
Contributor

@zcbenz zcbenz left a comment

Choose a reason for hiding this comment

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

Thanks!

@zcbenz zcbenz added the commit-queue Add this label to land a pull request using GitHub Actions. label Mar 30, 2024
@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 Mar 31, 2024
@nodejs-github-bot
Copy link
Collaborator

Commit Queue failed
- Loading data for nodejs/node/pull/52284
✔  Done loading data for nodejs/node/pull/52284
----------------------------------- PR info ------------------------------------
Title      doc: fix email pattern to be wrapped with `<<` instead of single `<` (#52284)
   ⚠  Could not retrieve the email or name of the PR author's from user's GitHub profile!
Branch     rluvaton:fix-email -> nodejs:main
Labels     doc, fast-track
Commits    1
 - doc: fix email pattern to be wrapped with `<<` instead of single `<`
Committers 1
 - Raz Luvaton <[email protected]>
PR-URL: https://github.com/nodejs/node/pull/52284
------------------------------ Generated metadata ------------------------------
PR-URL: https://github.com/nodejs/node/pull/52284
--------------------------------------------------------------------------------
   ℹ  This PR was created on Sat, 30 Mar 2024 21:33:34 GMT
   ✘  Approvals: 0
   ℹ  This PR is being fast-tracked
   ✔  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/8494208088

@VoltrexKeyva
Copy link
Member

@zcbenz note that fast-tracking PRs require 2 upvotes (2 thumbs up reactions on GitHub Actions bot's comment).

Documented here:

The pull request can be fast-tracked if two collaborators approve the fast-tracking request. To land, the pull request itself still needs two collaborator approvals and a passing CI.

@benjamingr benjamingr 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 Mar 31, 2024
@rluvaton rluvaton added the commit-queue-squash Add this label to instruct the Commit Queue to squash all the PR commits into the first one. label Mar 31, 2024
@nodejs-github-bot nodejs-github-bot removed the commit-queue Add this label to land a pull request using GitHub Actions. label Mar 31, 2024
@nodejs-github-bot nodejs-github-bot merged commit d9aa33f into nodejs:main Mar 31, 2024
@nodejs-github-bot
Copy link
Collaborator

Landed in d9aa33f

@rluvaton rluvaton deleted the fix-email branch April 2, 2024 13:04
marco-ippolito pushed a commit that referenced this pull request May 2, 2024
PR-URL: #52284
Reviewed-By: Mohammed Keyvanzadeh <[email protected]>
Reviewed-By: Yagiz Nizipli <[email protected]>
Reviewed-By: Benjamin Gruenbaum <[email protected]>
marco-ippolito pushed a commit that referenced this pull request May 3, 2024
PR-URL: #52284
Reviewed-By: Mohammed Keyvanzadeh <[email protected]>
Reviewed-By: Yagiz Nizipli <[email protected]>
Reviewed-By: Benjamin Gruenbaum <[email protected]>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
commit-queue-squash Add this label to instruct the Commit Queue to squash all the PR commits into the first one. doc Issues and PRs related to the documentations. fast-track PRs that do not need to wait for 48 hours to land.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

6 participants