Skip to content

meta: move Forrest Norvell to Collaborator Emeriti list - #27437

Closed
Trott wants to merge 1 commit into
nodejs:masterfrom
Trott:move-to-emeritus
Closed

meta: move Forrest Norvell to Collaborator Emeriti list#27437
Trott wants to merge 1 commit into
nodejs:masterfrom
Trott:move-to-emeritus

Conversation

@Trott

Copy link
Copy Markdown
Member

Consulted with Forrest. He thought this was appropriate as long as it
doesn't affect his ability to be on the Moderation Team.

Checklist
  • make -j4 test (UNIX), or vcbuild test (Windows) passes
  • documentation is changed or added
  • commit message follows commit guidelines

Consulted with Forrest. He thought this was appropriate as long as it
doesn't affect his ability to be on the Moderation Team.
@nodejs-github-bot

Copy link
Copy Markdown
Collaborator

@nodejs-github-botnodejs-github-bot added the doc Issues and PRs related to the documentations. label Apr 27, 2019
@Trott
Trott requested a review from othiym23April 27, 2019 01:43
@TrottTrott added the author ready PRs that have at least one approval, no pending requests for changes, and a CI started. label Apr 27, 2019

@othiym23othiym23 left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

Thanks for being a diligent steward of the process, @Trott! Maybe I'll return to actively contributing in the future!

@BridgeARBridgeAR left a comment

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

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

@othiym23 thanks for all your work!

@danbev

Copy link
Copy Markdown
Contributor

Landed in d33fddb.

@danbevdanbev closed this Apr 30, 2019
danbev pushed a commit that referenced this pull request Apr 30, 2019
Consulted with Forrest. He thought this was appropriate as long as it
doesn't affect his ability to be on the Moderation Team.
PR-URL: #27437
Reviewed-By: Gireesh Punathil <gpunathi@in.ibm.com>
Reviewed-By: Forrest L Norvell <ogd@aoaioxxysz.net>
Reviewed-By: Ruben Bridgewater <ruben@bridgewater.de>
Reviewed-By: Colin Ihrig <cjihrig@gmail.com>
Reviewed-By: Benjamin Gruenbaum <benjamingr@gmail.com>
Reviewed-By: Weijia Wang <starkwang@126.com>
targos pushed a commit that referenced this pull request Apr 30, 2019
Consulted with Forrest. He thought this was appropriate as long as it
doesn't affect his ability to be on the Moderation Team.
PR-URL: #27437
Reviewed-By: Gireesh Punathil <gpunathi@in.ibm.com>
Reviewed-By: Forrest L Norvell <ogd@aoaioxxysz.net>
Reviewed-By: Ruben Bridgewater <ruben@bridgewater.de>
Reviewed-By: Colin Ihrig <cjihrig@gmail.com>
Reviewed-By: Benjamin Gruenbaum <benjamingr@gmail.com>
Reviewed-By: Weijia Wang <starkwang@126.com>
@benjamingr

Copy link
Copy Markdown
Member

@Trott@danbev note that this landed but the actual GitHub team needs to be updated to reflect this change.

@Trott

Copy link
Copy Markdown
MemberAuthor

@Trott@danbev note that this landed but the actual GitHub team needs to be updated to reflect this change.

Yes, done!

Sign up for freeto join this conversation on GitHub. Already have an account? Sign in to comment

Labels

author readyPRs that have at least one approval, no pending requests for changes, and a CI started.docIssues and PRs related to the documentations.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

9 participants

@Trott@nodejs-github-bot@danbev@benjamingr@othiym23@cjihrig@gireeshpunathil@BridgeAR@starkwang