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: add Mesteery to triagers #39887

Closed
wants to merge 1 commit into from
Closed

Conversation

Mesteery
Copy link
Contributor

I would like to join the Triagers team, to be more useful, help manage issues and get more involved in Node.js.
I hereby declare that I have read and understood the Code of Conduct and will adhere to that.

@nodejs-github-bot nodejs-github-bot added the doc Issues and PRs related to the documentations. label Aug 25, 2021
@VoltrexKeyva
Copy link
Member

VoltrexKeyva commented Aug 26, 2021

Although the changes are looking good, I suggest removing the @ from the commit message as you'll be notified by github notifs everytime this commit is either cherry-picked or pushed somewhere (Refs: #37667 (comment))

@Mesteery
Copy link
Contributor Author

Okay, thanks for the suggestion!

@Mesteery Mesteery changed the title doc: add @Mesteery to triagers doc: add Mesteery to triagers Aug 26, 2021
@gireeshpunathil gireeshpunathil added the commit-queue Add this label to land a pull request using GitHub Actions. label Aug 27, 2021
@github-actions github-actions bot removed the commit-queue Add this label to land a pull request using GitHub Actions. label Aug 27, 2021
@github-actions
Copy link
Contributor

Commit Queue failed
- Loading data for nodejs/node/pull/39887
✔  Done loading data for nodejs/node/pull/39887
----------------------------------- PR info ------------------------------------
Title      doc: add Mesteery to triagers (#39887)
Author     Mestery  (@Mesteery)
Branch     Mesteery:patch-2 -> nodejs:master
Labels     doc
Commits    1
 - doc: add Mesteery to triagers
Committers 1
 - Mestery 
PR-URL: https://github.com/nodejs/node/pull/39887
Reviewed-By: Antoine du Hamel 
Reviewed-By: Michaël Zasso 
Reviewed-By: Gireesh Punathil 
Reviewed-By: Rich Trott 
Reviewed-By: Luigi Pinca 
Reviewed-By: James M Snell 
------------------------------ Generated metadata ------------------------------
PR-URL: https://github.com/nodejs/node/pull/39887
Reviewed-By: Antoine du Hamel 
Reviewed-By: Michaël Zasso 
Reviewed-By: Gireesh Punathil 
Reviewed-By: Rich Trott 
Reviewed-By: Luigi Pinca 
Reviewed-By: James M Snell 
--------------------------------------------------------------------------------
   ℹ  This PR was created on Wed, 25 Aug 2021 19:53:20 GMT
   ✔  Approvals: 6
   ✔  - Antoine du Hamel (@aduh95) (TSC): https://github.com/nodejs/node/pull/39887#pullrequestreview-738862440
   ✔  - Michaël Zasso (@targos) (TSC): https://github.com/nodejs/node/pull/39887#pullrequestreview-738866588
   ✔  - Gireesh Punathil (@gireeshpunathil) (TSC): https://github.com/nodejs/node/pull/39887#pullrequestreview-738990556
   ✔  - Rich Trott (@Trott) (TSC): https://github.com/nodejs/node/pull/39887#pullrequestreview-739037745
   ✔  - Luigi Pinca (@lpinca): https://github.com/nodejs/node/pull/39887#pullrequestreview-739407590
   ✔  - James M Snell (@jasnell) (TSC): https://github.com/nodejs/node/pull/39887#pullrequestreview-739507240
   ✖  This PR needs to wait 10 more hours to land
   ✔  Last GitHub Actions successful
   ℹ  Green GitHub Actions CI is sufficient
--------------------------------------------------------------------------------
   ✔  Aborted `git node land` session in /home/runner/work/node/node/.ncu
https://github.com/nodejs/node/actions/runs/1173693538

@github-actions github-actions bot added the commit-queue-failed An error occurred while landing this pull request using GitHub Actions. label Aug 27, 2021
Copy link
Member

@mhdawson mhdawson left a comment

Choose a reason for hiding this comment

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

LGTM

@targos targos 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 Aug 27, 2021
@github-actions github-actions bot removed the commit-queue Add this label to land a pull request using GitHub Actions. label Aug 27, 2021
@github-actions
Copy link
Contributor

Landed in b5ee5b5...e5670f4

@github-actions github-actions bot closed this Aug 27, 2021
nodejs-github-bot pushed a commit that referenced this pull request Aug 27, 2021
PR-URL: #39887
Reviewed-By: Antoine du Hamel <duhamelantoine1995@gmail.com>
Reviewed-By: Michaël Zasso <targos@protonmail.com>
Reviewed-By: Gireesh Punathil <gpunathi@in.ibm.com>
Reviewed-By: Rich Trott <rtrott@gmail.com>
Reviewed-By: Luigi Pinca <luigipinca@gmail.com>
Reviewed-By: James M Snell <jasnell@gmail.com>
Reviewed-By: Michael Dawson <midawson@redhat.com>
Reviewed-By: Darshan Sen <raisinten@gmail.com>
Reviewed-By: Zijian Liu <lxxyxzj@gmail.com>
@Mesteery Mesteery deleted the patch-2 branch August 27, 2021 21:13
@gireeshpunathil
Copy link
Member

sent invitation to @Mesteery to join @nodejs/issue-triage team

@Mesteery
Copy link
Contributor Author

Invitation accepted, thank you!

@himadriganguly
Copy link
Contributor

Welcome @Mesteery to the team.

targos pushed a commit that referenced this pull request Sep 6, 2021
PR-URL: #39887
Reviewed-By: Antoine du Hamel <duhamelantoine1995@gmail.com>
Reviewed-By: Michaël Zasso <targos@protonmail.com>
Reviewed-By: Gireesh Punathil <gpunathi@in.ibm.com>
Reviewed-By: Rich Trott <rtrott@gmail.com>
Reviewed-By: Luigi Pinca <luigipinca@gmail.com>
Reviewed-By: James M Snell <jasnell@gmail.com>
Reviewed-By: Michael Dawson <midawson@redhat.com>
Reviewed-By: Darshan Sen <raisinten@gmail.com>
Reviewed-By: Zijian Liu <lxxyxzj@gmail.com>
targos pushed a commit that referenced this pull request Sep 6, 2021
PR-URL: #39887
Reviewed-By: Antoine du Hamel <duhamelantoine1995@gmail.com>
Reviewed-By: Michaël Zasso <targos@protonmail.com>
Reviewed-By: Gireesh Punathil <gpunathi@in.ibm.com>
Reviewed-By: Rich Trott <rtrott@gmail.com>
Reviewed-By: Luigi Pinca <luigipinca@gmail.com>
Reviewed-By: James M Snell <jasnell@gmail.com>
Reviewed-By: Michael Dawson <midawson@redhat.com>
Reviewed-By: Darshan Sen <raisinten@gmail.com>
Reviewed-By: Zijian Liu <lxxyxzj@gmail.com>
@targos targos mentioned this pull request Sep 6, 2021
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
doc Issues and PRs related to the documentations.
Projects
None yet
Development

Successfully merging this pull request may close these issues.