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.

Sign up for GitHub

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

Jump to bottom

meta: move one or more collaborators to emeritus #49264

Merged
merged 1 commit into from
Aug 23, 2023
Merged

meta: move one or more collaborators to emeritus #49264

merged 1 commit into from
Aug 23, 2023

Conversation

nodejs-github-bot
Copy link
Collaborator

This PR was generated by tools/find-inactive-collaborators.yml.

@nodejs-github-bot nodejs-github-bot added meta Issues and PRs related to the general management of the project. doc Issues and PRs related to the documentations. labels Aug 21, 2023
debadree25
debadree25 requested changes Aug 21, 2023
View reviewed changes
Copy link
Member

@debadree25 debadree25 left a comment

Choose a reason for hiding this comment

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

Doesn't seem right adding a request changes

debadree25
debadree25 reviewed Aug 21, 2023
View reviewed changes
README.md Outdated Show resolved Hide resolved
@atlowChemi atlowChemi force-pushed the actions/inactive-collaborators branch from 58d2559 to 2a25d26 Compare August 21, 2023 07:42
@aduh95
Copy link
Contributor

aduh95 commented Aug 21, 2023

@Leko FYI

Leko
Leko approved these changes Aug 21, 2023
View reviewed changes
Copy link
Contributor

@Leko Leko left a comment

Choose a reason for hiding this comment

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

Thank you for notifying. I would like to be an active contributor when I can devote more time to Node.js again.

@aduh95 aduh95 requested a review from debadree25 August 21, 2023 08:58
@aduh95 aduh95 added author ready PRs that have at least one approval, no pending requests for changes, and a CI started. commit-queue Add this label to land a pull request using GitHub Actions. labels Aug 21, 2023
@debadree25 debadree25 added author ready PRs that have at least one approval, no pending requests for changes, and a CI started. commit-queue Add this label to land a pull request using GitHub Actions. labels Aug 21, 2023
@nodejs-github-bot nodejs-github-bot removed the commit-queue Add this label to land a pull request using GitHub Actions. label Aug 23, 2023
@nodejs-github-bot nodejs-github-bot merged commit 0d080a2 into main Aug 23, 2023
23 checks passed
@nodejs-github-bot nodejs-github-bot deleted the actions/inactive-collaborators branch August 23, 2023 04:21
@nodejs-github-bot
Copy link
Collaborator Author

Landed in 0d080a2

UlisesGascon pushed a commit that referenced this pull request Sep 10, 2023
@nodejs-github-bot @UlisesGascon
meta: move one or more collaborators to emeritus
81ff68c 
PR-URL: #49264
Reviewed-By: Debadree Chatterjee <debadree333@gmail.com>
Reviewed-By: Antoine du Hamel <duhamelantoine1995@gmail.com>
Reviewed-By: Shingo Inoue <leko.noor@gmail.com>
Reviewed-By: Luigi Pinca <luigipinca@gmail.com>
@UlisesGascon UlisesGascon mentioned this pull request Sep 10, 2023
Merged
@Trott
Copy link
Member

Trott commented Oct 11, 2023 •
edited
Loading

This landed but nobody on @nodejs/tsc followed up with the offboarding tasks at https://github.com/nodejs/node/blob/main/doc/contributing/offboarding.md. I'll do that now. (Pinging TSC as a reminder when this or something similar comes up in the future. In the past, I've usually been the one to do those things, but I've been following the repo less closely for the last few months.)

EDIT: All done.

tniessen added a commit to tniessen/node that referenced this pull request Oct 11, 2023
@tniessen
meta: ping TSC for offboarding
7d97eb8 
Refs: nodejs#49264
@tniessen tniessen mentioned this pull request Oct 11, 2023
Merged
nodejs-github-bot pushed a commit that referenced this pull request Oct 13, 2023
@tniessen
meta: ping TSC for offboarding
bac85be 
Refs: #49264
PR-URL: #50147
Reviewed-By: Richard Lau <rlau@redhat.com>
Reviewed-By: Yagiz Nizipli <yagiz@nizipli.com>
Reviewed-By: Michael Dawson <midawson@redhat.com>
Reviewed-By: Luigi Pinca <luigipinca@gmail.com>
Reviewed-By: Tierney Cyren <hello@bnb.im>
Reviewed-By: Rich Trott <rtrott@gmail.com>
alexfernandez pushed a commit to alexfernandez/node that referenced this pull request Nov 1, 2023
@nodejs-github-bot @alexfernandez
meta: move one or more collaborators to emeritus
0987e8a 
PR-URL: nodejs#49264
Reviewed-By: Debadree Chatterjee <debadree333@gmail.com>
Reviewed-By: Antoine du Hamel <duhamelantoine1995@gmail.com>
Reviewed-By: Shingo Inoue <leko.noor@gmail.com>
Reviewed-By: Luigi Pinca <luigipinca@gmail.com>
alexfernandez pushed a commit to alexfernandez/node that referenced this pull request Nov 1, 2023
@tniessen @alexfernandez
meta: ping TSC for offboarding
3912933 
Refs: nodejs#49264
PR-URL: nodejs#50147
Reviewed-By: Richard Lau <rlau@redhat.com>
Reviewed-By: Yagiz Nizipli <yagiz@nizipli.com>
Reviewed-By: Michael Dawson <midawson@redhat.com>
Reviewed-By: Luigi Pinca <luigipinca@gmail.com>
Reviewed-By: Tierney Cyren <hello@bnb.im>
Reviewed-By: Rich Trott <rtrott@gmail.com>
targos pushed a commit that referenced this pull request Nov 11, 2023
@tniessen @targos
meta: ping TSC for offboarding
27e957c 
Refs: #49264
PR-URL: #50147
Reviewed-By: Richard Lau <rlau@redhat.com>
Reviewed-By: Yagiz Nizipli <yagiz@nizipli.com>
Reviewed-By: Michael Dawson <midawson@redhat.com>
Reviewed-By: Luigi Pinca <luigipinca@gmail.com>
Reviewed-By: Tierney Cyren <hello@bnb.im>
Reviewed-By: Rich Trott <rtrott@gmail.com>
targos pushed a commit that referenced this pull request Nov 27, 2023
@nodejs-github-bot @targos
meta: move one or more collaborators to emeritus
155a275 
PR-URL: #49264
Reviewed-By: Debadree Chatterjee <debadree333@gmail.com>
Reviewed-By: Antoine du Hamel <duhamelantoine1995@gmail.com>
Reviewed-By: Shingo Inoue <leko.noor@gmail.com>
Reviewed-By: Luigi Pinca <luigipinca@gmail.com>
targos pushed a commit that referenced this pull request Nov 27, 2023
@tniessen @targos
meta: ping TSC for offboarding
5b9575d 
Refs: #49264
PR-URL: #50147
Reviewed-By: Richard Lau <rlau@redhat.com>
Reviewed-By: Yagiz Nizipli <yagiz@nizipli.com>
Reviewed-By: Michael Dawson <midawson@redhat.com>
Reviewed-By: Luigi Pinca <luigipinca@gmail.com>
Reviewed-By: Tierney Cyren <hello@bnb.im>
Reviewed-By: Rich Trott <rtrott@gmail.com>
@targos targos mentioned this pull request Nov 28, 2023
Merged
sercher added a commit to sercher/graaljs that referenced this pull request Apr 25, 2024
@sercher
meta: move one or more collaborators to emeritus
82eb82d 
PR-URL: nodejs/node#49264
Reviewed-By: Debadree Chatterjee <debadree333@gmail.com>
Reviewed-By: Antoine du Hamel <duhamelantoine1995@gmail.com>
Reviewed-By: Shingo Inoue <leko.noor@gmail.com>
Reviewed-By: Luigi Pinca <luigipinca@gmail.com>
sercher added a commit to sercher/graaljs that referenced this pull request Apr 25, 2024
@sercher
meta: ping TSC for offboarding
587cf5e 
Refs: nodejs/node#49264
PR-URL: nodejs/node#50147
Reviewed-By: Richard Lau <rlau@redhat.com>
Reviewed-By: Yagiz Nizipli <yagiz@nizipli.com>
Reviewed-By: Michael Dawson <midawson@redhat.com>
Reviewed-By: Luigi Pinca <luigipinca@gmail.com>
Reviewed-By: Tierney Cyren <hello@bnb.im>
Reviewed-By: Rich Trott <rtrott@gmail.com>
sercher added a commit to sercher/graaljs that referenced this pull request Apr 25, 2024
@sercher
meta: move one or more collaborators to emeritus
4fdb81b 
PR-URL: nodejs/node#49264
Reviewed-By: Debadree Chatterjee <debadree333@gmail.com>
Reviewed-By: Antoine du Hamel <duhamelantoine1995@gmail.com>
Reviewed-By: Shingo Inoue <leko.noor@gmail.com>
Reviewed-By: Luigi Pinca <luigipinca@gmail.com>
sercher added a commit to sercher/graaljs that referenced this pull request Apr 25, 2024
@sercher
meta: ping TSC for offboarding
a1b3133 
Refs: nodejs/node#49264
PR-URL: nodejs/node#50147
Reviewed-By: Richard Lau <rlau@redhat.com>
Reviewed-By: Yagiz Nizipli <yagiz@nizipli.com>
Reviewed-By: Michael Dawson <midawson@redhat.com>
Reviewed-By: Luigi Pinca <luigipinca@gmail.com>
Reviewed-By: Tierney Cyren <hello@bnb.im>
Reviewed-By: Rich Trott <rtrott@gmail.com>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@atlowChemi atlowChemi atlowChemi left review comments

@debadree25 debadree25 debadree25 approved these changes

@aduh95 aduh95 aduh95 approved these changes

@lpinca lpinca lpinca approved these changes

@Leko Leko Leko approved these changes

Assignees
No one assigned
Labels
author ready PRs that have at least one approval, no pending requests for changes, and a CI started. doc Issues and PRs related to the documentations. meta Issues and PRs related to the general management of the project.
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
7 participants
@nodejs-github-bot @aduh95 @Trott @Leko @lpinca @debadree25 @atlowChemi