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 RaisinTen to emeritus and remove from strategic initiatives #51411

Merged
merged 1 commit into from
Jan 12, 2024
Merged

meta: move RaisinTen to emeritus and remove from strategic initiatives #51411

merged 1 commit into from
Jan 12, 2024

Conversation

RaisinTen
Copy link
Contributor

No description provided.

@nodejs-github-bot
Copy link
Collaborator

Review requested:

  • @nodejs/tsc

@nodejs-github-bot nodejs-github-bot added the doc Issues and PRs related to the documentations. label Jan 9, 2024
mcollina
mcollina approved these changes Jan 9, 2024
View reviewed changes
Copy link
Member

@mcollina mcollina left a comment

Choose a reason for hiding this comment

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

lgtm

@RaisinTen RaisinTen force-pushed the meta-move-RaisinTen-to-emeritus-and-remove-from-strategic-initiatives branch from c28b8b3 to b4f78a3 Compare January 9, 2024 11:22
RafaelGSS
RafaelGSS approved these changes Jan 9, 2024
View reviewed changes
Copy link
Member

@RafaelGSS RafaelGSS left a comment

Choose a reason for hiding this comment

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

Thanks for all efforts!

aduh95
aduh95 approved these changes Jan 9, 2024
View reviewed changes
Copy link
Contributor

@aduh95 aduh95 left a comment

Choose a reason for hiding this comment

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

Thanks for all the work, I hope I'll see you around!

tniessen
tniessen approved these changes Jan 9, 2024
View reviewed changes
Copy link
Member

@tniessen tniessen 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 @RaisinTen!

bnb
bnb approved these changes Jan 9, 2024
View reviewed changes
Copy link
Contributor

@bnb bnb left a comment

Choose a reason for hiding this comment

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

Appreciate you, Darshan

mhdawson
mhdawson approved these changes Jan 9, 2024
View reviewed changes
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. Sorry to see you go, thanks for all of the work over the years.

@cola119 cola119 added the meta Issues and PRs related to the general management of the project. label Jan 10, 2024
cola119
cola119 approved these changes Jan 10, 2024
View reviewed changes
Copy link
Member

@cola119 cola119 left a comment

Choose a reason for hiding this comment

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

❤️

MoLow
MoLow approved these changes Jan 10, 2024
View reviewed changes
Copy link
Member

@MoLow MoLow left a comment

Choose a reason for hiding this comment

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

Thanks for the hard work 🙏

marco-ippolito
marco-ippolito approved these changes Jan 10, 2024
View reviewed changes
Copy link
Member

@marco-ippolito marco-ippolito left a comment

Choose a reason for hiding this comment

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

Thanks for you work! 🫂

joyeecheung
joyeecheung approved these changes Jan 10, 2024
View reviewed changes
Copy link
Member

@joyeecheung joyeecheung left a comment

Choose a reason for hiding this comment

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

Hope to see you around again

benjamingr
benjamingr approved these changes Jan 10, 2024
View reviewed changes
Copy link
Member

@benjamingr benjamingr 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 the work! (fwiw, you are not moving to emeritus but to "regular" vs "voting" TSC member)

@mhdawson
Copy link
Member

To whoever gets to merge this, there is an offboarding list and we should either leave this open until that list is complete or maybe create an issuein the TSC repo to do the offboarding.

@RaisinTen
Copy link
Contributor Author

fwiw, you are not moving to emeritus but to "regular" vs "voting" TSC member

@benjamingr I just double-checked. My name is in the TSC emeriti members list, not the TSC regular members list.

@tniessen tniessen 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. and removed commit-queue Add this label to land a pull request using GitHub Actions. labels Jan 11, 2024
@aduh95 aduh95 added the commit-queue Add this label to land a pull request using GitHub Actions. label Jan 12, 2024
@nodejs-github-bot nodejs-github-bot removed the commit-queue Add this label to land a pull request using GitHub Actions. label Jan 12, 2024
@nodejs-github-bot nodejs-github-bot merged commit b7ca80f into nodejs:main Jan 12, 2024
25 checks passed
@nodejs-github-bot
Copy link
Collaborator

Landed in b7ca80f

@RaisinTen RaisinTen deleted the meta-move-RaisinTen-to-emeritus-and-remove-from-strategic-initiatives branch January 12, 2024 09:00
This was referenced Jan 12, 2024
Merged
Merged
Closed
richardlau pushed a commit to nodejs/email that referenced this pull request Jan 12, 2024
@aduh95
Remove RaisinTen from TSC and crypto-export mailing list (#236)
fc19a1a 
Refs: nodejs/node#51411
marco-ippolito pushed a commit to marco-ippolito/node that referenced this pull request Jan 12, 2024
@RaisinTen @marco-ippolito
meta: move RaisinTen to emeritus and remove from strategic initiatives
b5528d3 
Signed-off-by: Darshan Sen <[email protected]>
PR-URL: nodejs#51411
Reviewed-By: Matteo Collina <[email protected]>
Reviewed-By: Rafael Gonzaga <[email protected]>
Reviewed-By: Antoine du Hamel <[email protected]>
Reviewed-By: Tobias Nießen <[email protected]>
Reviewed-By: Tierney Cyren <[email protected]>
Reviewed-By: Michael Dawson <[email protected]>
Reviewed-By: Kohei Ueno <[email protected]>
Reviewed-By: Moshe Atlow <[email protected]>
Reviewed-By: Marco Ippolito <[email protected]>
Reviewed-By: Joyee Cheung <[email protected]>
Reviewed-By: Benjamin Gruenbaum <[email protected]>
Reviewed-By: Luigi Pinca <[email protected]>
Medhansh404 pushed a commit to Medhansh404/node that referenced this pull request Jan 19, 2024
@RaisinTen @Medhansh404
meta: move RaisinTen to emeritus and remove from strategic initiatives
204c3a3 
Signed-off-by: Darshan Sen <[email protected]>
PR-URL: nodejs#51411
Reviewed-By: Matteo Collina <[email protected]>
Reviewed-By: Rafael Gonzaga <[email protected]>
Reviewed-By: Antoine du Hamel <[email protected]>
Reviewed-By: Tobias Nießen <[email protected]>
Reviewed-By: Tierney Cyren <[email protected]>
Reviewed-By: Michael Dawson <[email protected]>
Reviewed-By: Kohei Ueno <[email protected]>
Reviewed-By: Moshe Atlow <[email protected]>
Reviewed-By: Marco Ippolito <[email protected]>
Reviewed-By: Joyee Cheung <[email protected]>
Reviewed-By: Benjamin Gruenbaum <[email protected]>
Reviewed-By: Luigi Pinca <[email protected]>
targos pushed a commit that referenced this pull request Feb 15, 2024
@RaisinTen @targos
meta: move RaisinTen to emeritus and remove from strategic initiatives
015c4dc 
Signed-off-by: Darshan Sen <[email protected]>
PR-URL: #51411
Reviewed-By: Matteo Collina <[email protected]>
Reviewed-By: Rafael Gonzaga <[email protected]>
Reviewed-By: Antoine du Hamel <[email protected]>
Reviewed-By: Tobias Nießen <[email protected]>
Reviewed-By: Tierney Cyren <[email protected]>
Reviewed-By: Michael Dawson <[email protected]>
Reviewed-By: Kohei Ueno <[email protected]>
Reviewed-By: Moshe Atlow <[email protected]>
Reviewed-By: Marco Ippolito <[email protected]>
Reviewed-By: Joyee Cheung <[email protected]>
Reviewed-By: Benjamin Gruenbaum <[email protected]>
Reviewed-By: Luigi Pinca <[email protected]>
@marco-ippolito marco-ippolito mentioned this pull request Mar 1, 2024
Merged
richardlau pushed a commit that referenced this pull request Mar 25, 2024
@RaisinTen @richardlau
meta: move RaisinTen to emeritus and remove from strategic initiatives
3c7ae95 
Signed-off-by: Darshan Sen <[email protected]>
PR-URL: #51411
Reviewed-By: Matteo Collina <[email protected]>
Reviewed-By: Rafael Gonzaga <[email protected]>
Reviewed-By: Antoine du Hamel <[email protected]>
Reviewed-By: Tobias Nießen <[email protected]>
Reviewed-By: Tierney Cyren <[email protected]>
Reviewed-By: Michael Dawson <[email protected]>
Reviewed-By: Kohei Ueno <[email protected]>
Reviewed-By: Moshe Atlow <[email protected]>
Reviewed-By: Marco Ippolito <[email protected]>
Reviewed-By: Joyee Cheung <[email protected]>
Reviewed-By: Benjamin Gruenbaum <[email protected]>
Reviewed-By: Luigi Pinca <[email protected]>
richardlau pushed a commit that referenced this pull request Mar 25, 2024
@RaisinTen @richardlau
meta: move RaisinTen to emeritus and remove from strategic initiatives
4a8a012 
Signed-off-by: Darshan Sen <[email protected]>
PR-URL: #51411
Reviewed-By: Matteo Collina <[email protected]>
Reviewed-By: Rafael Gonzaga <[email protected]>
Reviewed-By: Antoine du Hamel <[email protected]>
Reviewed-By: Tobias Nießen <[email protected]>
Reviewed-By: Tierney Cyren <[email protected]>
Reviewed-By: Michael Dawson <[email protected]>
Reviewed-By: Kohei Ueno <[email protected]>
Reviewed-By: Moshe Atlow <[email protected]>
Reviewed-By: Marco Ippolito <[email protected]>
Reviewed-By: Joyee Cheung <[email protected]>
Reviewed-By: Benjamin Gruenbaum <[email protected]>
Reviewed-By: Luigi Pinca <[email protected]>
@richardlau richardlau mentioned this pull request Mar 25, 2024
Merged
Johnhvy pushed a commit to Johnhvy/NodeJS-Email that referenced this pull request Jul 16, 2024
@aduh95
Remove RaisinTen from TSC and crypto-export mailing list (#236)
6bcfa2b 
Refs: nodejs/node#51411
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@marco-ippolito marco-ippolito marco-ippolito approved these changes

@RafaelGSS RafaelGSS RafaelGSS approved these changes

@cola119 cola119 cola119 approved these changes

@aduh95 aduh95 aduh95 approved these changes

@mhdawson mhdawson mhdawson approved these changes

@MoLow MoLow MoLow approved these changes

@joyeecheung joyeecheung joyeecheung approved these changes

@tniessen tniessen tniessen approved these changes

@lpinca lpinca lpinca approved these changes

@benjamingr benjamingr benjamingr approved these changes

@bnb bnb bnb approved these changes

@mcollina mcollina mcollina 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.