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

doc: move gibfahn to emeritus #23238

Closed
wants to merge 1 commit into from
Closed

doc: move gibfahn to emeritus #23238

wants to merge 1 commit into from

Conversation

gibfahn
Copy link
Member

@gibfahn gibfahn commented Oct 3, 2018

I should have done this a while back, but until I am able to contribute to Node again it doesn't make sense for me to continue to be on the TSC.

Looking forward to being able to contribute more once I can, hopefully soon 🤞!

@nodejs-github-bot
Copy link
Collaborator

@gibfahn build started: https://ci.nodejs.org/blue/organizations/jenkins/node-test-pull-request-lite-pipeline/detail/node-test-pull-request-lite-pipeline/1111/pipeline

@nodejs-github-bot nodejs-github-bot added the doc Issues and PRs related to the documentations. label Oct 3, 2018
tniessen
tniessen approved these changes Oct 3, 2018
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.

Looking forward to it!

joyeecheung
joyeecheung approved these changes Oct 3, 2018
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.

Looking forward to seeing you back!

gibfahn added a commit to gibfahn/email that referenced this pull request Oct 3, 2018
@gibfahn
Remove gibfahn from TSC mailing list
b33537b 
Refs: nodejs/node#23238
@gibfahn gibfahn mentioned this pull request Oct 3, 2018
Merged
@benjamingr
Copy link
Member

benjamingr commented Oct 3, 2018 •
edited
Loading

Thank you for your contribution to the project - it has made a real difference to a lot of people.

I hope to see you back here in the future.

BridgeAR
BridgeAR approved these changes Oct 3, 2018
View reviewed changes
Copy link
Member

@BridgeAR BridgeAR left a comment

Choose a reason for hiding this comment

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

Thanks a lot for all your work! Looking forward to seeing you back!

mcollina
mcollina approved these changes Oct 3, 2018
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.

Thank you Gibson! I hope you will be able to be more involved in the future!

@Trott
Copy link
Member

Trott commented Oct 3, 2018

Thanks for everything, Gibson! Hope to see you around soon!

FWIW, when this lands, might be good to insert TSC into the commit message subject line to be clear that we're talking about TSC Emeritus, not Collaborator Emeritus.

@Trott
Copy link
Member

Trott commented Oct 3, 2018

Adding fast-track since there seems to be little reason to delay someone moving themself in this way. 👍 here to fast-track.

@Trott Trott added the fast-track PRs that do not need to wait for 48 hours to land. label Oct 3, 2018
jasnell
jasnell approved these changes Oct 3, 2018
View reviewed changes
Copy link
Member

@jasnell jasnell left a comment

Choose a reason for hiding this comment

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

@gibfahn ... definitely hope the path is soon cleared for you to continue contributing! Definitely hope that you're enjoying the new role tho!

Trott pushed a commit to Trott/io.js that referenced this pull request Oct 3, 2018
@gibfahn @Trott
doc: move gibfahn to TSC Emeritus
2ebdba1 
PR-URL: nodejs#23238
Reviewed-By: Tobias Nießen <[email protected]>
Reviewed-By: Joyee Cheung <[email protected]>
Reviewed-By: Gireesh Punathil <[email protected]>
Reviewed-By: Benjamin Gruenbaum <[email protected]>
Reviewed-By: Ruben Bridgewater <[email protected]>
Reviewed-By: Luigi Pinca <[email protected]>
Reviewed-By: Matteo Collina <[email protected]>
Reviewed-By: Colin Ihrig <[email protected]>
Reviewed-By: Anna Henningsen <[email protected]>
Reviewed-By: Richard Lau <[email protected]>
Reviewed-By: James M Snell <[email protected]>
@Trott
Copy link
Member

Trott commented Oct 3, 2018

Landed in 2ebdba1

@Trott Trott closed this Oct 3, 2018
@gibfahn gibfahn deleted the gib_to_tsc_emeritus branch October 3, 2018 17:40
targos pushed a commit that referenced this pull request Oct 4, 2018
@gibfahn @targos
doc: move gibfahn to TSC Emeritus
6912376 
PR-URL: #23238
Reviewed-By: Tobias Nießen <[email protected]>
Reviewed-By: Joyee Cheung <[email protected]>
Reviewed-By: Gireesh Punathil <[email protected]>
Reviewed-By: Benjamin Gruenbaum <[email protected]>
Reviewed-By: Ruben Bridgewater <[email protected]>
Reviewed-By: Luigi Pinca <[email protected]>
Reviewed-By: Matteo Collina <[email protected]>
Reviewed-By: Colin Ihrig <[email protected]>
Reviewed-By: Anna Henningsen <[email protected]>
Reviewed-By: Richard Lau <[email protected]>
Reviewed-By: James M Snell <[email protected]>
mhdawson pushed a commit to nodejs/email that referenced this pull request Oct 4, 2018
@gibfahn @mhdawson
Remove gibfahn from TSC mailing list
1450da9 
PR-URL: #114
Refs: nodejs/node#23238
Reviewed-By: Joyee Cheung <[email protected]>
Reviewed-By: Matteo Collina <[email protected]>
Reviewed-By: Rich Trott <[email protected]>
Reviewed-By: Michael Dawson <[email protected]>
@targos targos mentioned this pull request Oct 7, 2018
Merged
This was referenced Oct 10, 2018
Closed
Closed
Closed
This was referenced Oct 11, 2018
Closed
Closed
Closed
Closed
Closed
Closed
Closed
jasnell pushed a commit that referenced this pull request Oct 17, 2018
@gibfahn @jasnell
doc: move gibfahn to TSC Emeritus
171fe79 
PR-URL: #23238
Reviewed-By: Tobias Nießen <[email protected]>
Reviewed-By: Joyee Cheung <[email protected]>
Reviewed-By: Gireesh Punathil <[email protected]>
Reviewed-By: Benjamin Gruenbaum <[email protected]>
Reviewed-By: Ruben Bridgewater <[email protected]>
Reviewed-By: Luigi Pinca <[email protected]>
Reviewed-By: Matteo Collina <[email protected]>
Reviewed-By: Colin Ihrig <[email protected]>
Reviewed-By: Anna Henningsen <[email protected]>
Reviewed-By: Richard Lau <[email protected]>
Reviewed-By: James M Snell <[email protected]>
Johnhvy pushed a commit to Johnhvy/NodeJS-Email that referenced this pull request Jul 16, 2024
@gibfahn @Johnhvy
Remove gibfahn from TSC mailing list
c96b8f9 
PR-URL: nodejs/email#114
Refs: nodejs/node#23238
Reviewed-By: Joyee Cheung <[email protected]>
Reviewed-By: Matteo Collina <[email protected]>
Reviewed-By: Rich Trott <[email protected]>
Reviewed-By: Michael Dawson <[email protected]>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@BridgeAR BridgeAR BridgeAR approved these changes

@gireeshpunathil gireeshpunathil gireeshpunathil approved these changes

@richardlau richardlau richardlau approved these changes

@joyeecheung joyeecheung joyeecheung approved these changes

@tniessen tniessen tniessen approved these changes

@cjihrig cjihrig cjihrig approved these changes

@lpinca lpinca lpinca approved these changes

@benjamingr benjamingr benjamingr approved these changes

@addaleax addaleax addaleax approved these changes

@jasnell jasnell jasnell approved these changes

@mcollina mcollina mcollina approved these changes

Assignees
No one assigned
Labels
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
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.