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: remove collaborator self-nomination #57537

Merged
merged 1 commit into from
Mar 20, 2025
Merged

meta: remove collaborator self-nomination #57537

merged 1 commit into from
Mar 20, 2025
+0 −4

Conversation

Trott
Copy link
Member

@Trott Trott commented Mar 18, 2025 •
edited
Loading

Anyone with sufficient involvement in Node.js (such that becoming a collaborator would benefit Node.js) is going to have plenty of people to ask. The self-nomination process has been entirely superfluous the entire time I've been involved in Node.js. Simplify things and get rid of it.

This is based on my comment at https://github.com/nodejs/node/pull/57503/files#r1997694900. If there isn't consensus on this, then that's OK and we can just close it. I don't feel so strongly that I want to have a long back-and-forth about it. But if there is consensus, let's take this opportunity to simplify our governance.

Since this is a change to the Governance doc, I think it's appropriate to @-mention the TSC. @nodejs/tsc

@nodejs-github-bot
Copy link
Collaborator

Review requested:

  • @nodejs/tsc

@Trott Trott mentioned this pull request Mar 18, 2025
Closed
@nodejs-github-bot nodejs-github-bot added the meta Issues and PRs related to the general management of the project. label Mar 18, 2025
mcollina
mcollina approved these changes Mar 19, 2025
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

@aduh95
Copy link
Contributor

aduh95 commented Mar 19, 2025

This needs a rebase

@Trott
Copy link
Member Author

Trott commented Mar 19, 2025

This needs a rebase

Done.

@Trott
meta: remove collaborator self-nomination
7798046 
Anyone with sufficient involvement in Node.js such that becoming a
collaborator would benefit Node.js, is going to have plenty of people to
ask. The self-nomination process has been entirely superfluous the
entire time I've been involved in Node.js. Simplify things and get rid
of it.
@aduh95 aduh95 merged commit df883b0 into nodejs:main Mar 20, 2025
14 checks passed
@aduh95
Copy link
Contributor

aduh95 commented Mar 20, 2025

Landed in df883b0

@Trott Trott deleted the self-nom branch March 21, 2025 05:28
aduh95 pushed a commit that referenced this pull request Mar 23, 2025
@Trott @aduh95
meta: remove collaborator self-nomination
d0c96c4 
Anyone with sufficient involvement in Node.js such that becoming a
collaborator would benefit Node.js, is going to have plenty of people to
ask. The self-nomination process has been entirely superfluous the
entire time I've been involved in Node.js. Simplify things and get rid
of it.

PR-URL: #57537
Reviewed-By: Ruben Bridgewater <ruben@bridgewater.de>
Reviewed-By: Marco Ippolito <marcoippolito54@gmail.com>
Reviewed-By: Tobias Nießen <tniessen@tnie.de>
Reviewed-By: Michaël Zasso <targos@protonmail.com>
Reviewed-By: Yagiz Nizipli <yagiz@nizipli.com>
Reviewed-By: Luigi Pinca <luigipinca@gmail.com>
Reviewed-By: Ruy Adorno <ruy@vlt.sh>
Reviewed-By: Rafael Gonzaga <rafael.nunu@hotmail.com>
Reviewed-By: Jordan Harband <ljharb@gmail.com>
Reviewed-By: Ulises Gascón <ulisesgascongonzalez@gmail.com>
Reviewed-By: Trivikram Kamat <trivikr.dev@gmail.com>
Reviewed-By: Darshan Sen <raisinten@gmail.com>
Reviewed-By: Paolo Insogna <paolo@cowtech.it>
Reviewed-By: James M Snell <jasnell@gmail.com>
Reviewed-By: Edy Silva <edigleyssonsilva@gmail.com>
Reviewed-By: Matteo Collina <matteo.collina@gmail.com>
@github-actions github-actions bot mentioned this pull request Mar 30, 2025
Merged
RafaelGSS pushed a commit that referenced this pull request Apr 1, 2025
@Trott @RafaelGSS
meta: remove collaborator self-nomination
d5e1edf 
Anyone with sufficient involvement in Node.js such that becoming a
collaborator would benefit Node.js, is going to have plenty of people to
ask. The self-nomination process has been entirely superfluous the
entire time I've been involved in Node.js. Simplify things and get rid
of it.

PR-URL: #57537
Reviewed-By: Ruben Bridgewater <ruben@bridgewater.de>
Reviewed-By: Marco Ippolito <marcoippolito54@gmail.com>
Reviewed-By: Tobias Nießen <tniessen@tnie.de>
Reviewed-By: Michaël Zasso <targos@protonmail.com>
Reviewed-By: Yagiz Nizipli <yagiz@nizipli.com>
Reviewed-By: Luigi Pinca <luigipinca@gmail.com>
Reviewed-By: Ruy Adorno <ruy@vlt.sh>
Reviewed-By: Rafael Gonzaga <rafael.nunu@hotmail.com>
Reviewed-By: Jordan Harband <ljharb@gmail.com>
Reviewed-By: Ulises Gascón <ulisesgascongonzalez@gmail.com>
Reviewed-By: Trivikram Kamat <trivikr.dev@gmail.com>
Reviewed-By: Darshan Sen <raisinten@gmail.com>
Reviewed-By: Paolo Insogna <paolo@cowtech.it>
Reviewed-By: James M Snell <jasnell@gmail.com>
Reviewed-By: Edy Silva <edigleyssonsilva@gmail.com>
Reviewed-By: Matteo Collina <matteo.collina@gmail.com>
RafaelGSS pushed a commit that referenced this pull request Apr 1, 2025
@Trott @RafaelGSS
meta: remove collaborator self-nomination
ae6d18a 
Anyone with sufficient involvement in Node.js such that becoming a
collaborator would benefit Node.js, is going to have plenty of people to
ask. The self-nomination process has been entirely superfluous the
entire time I've been involved in Node.js. Simplify things and get rid
of it.

PR-URL: #57537
Reviewed-By: Ruben Bridgewater <ruben@bridgewater.de>
Reviewed-By: Marco Ippolito <marcoippolito54@gmail.com>
Reviewed-By: Tobias Nießen <tniessen@tnie.de>
Reviewed-By: Michaël Zasso <targos@protonmail.com>
Reviewed-By: Yagiz Nizipli <yagiz@nizipli.com>
Reviewed-By: Luigi Pinca <luigipinca@gmail.com>
Reviewed-By: Ruy Adorno <ruy@vlt.sh>
Reviewed-By: Rafael Gonzaga <rafael.nunu@hotmail.com>
Reviewed-By: Jordan Harband <ljharb@gmail.com>
Reviewed-By: Ulises Gascón <ulisesgascongonzalez@gmail.com>
Reviewed-By: Trivikram Kamat <trivikr.dev@gmail.com>
Reviewed-By: Darshan Sen <raisinten@gmail.com>
Reviewed-By: Paolo Insogna <paolo@cowtech.it>
Reviewed-By: James M Snell <jasnell@gmail.com>
Reviewed-By: Edy Silva <edigleyssonsilva@gmail.com>
Reviewed-By: Matteo Collina <matteo.collina@gmail.com>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@ljharb ljharb ljharb approved these changes

@mcollina mcollina mcollina approved these changes

@ShogunPanda ShogunPanda ShogunPanda approved these changes

@ruyadorno ruyadorno ruyadorno approved these changes

@jasnell jasnell jasnell approved these changes

@lpinca lpinca lpinca approved these changes

@anonrig anonrig anonrig approved these changes

@targos targos targos approved these changes

@tniessen tniessen tniessen approved these changes

@UlisesGascon UlisesGascon UlisesGascon approved these changes

@BridgeAR BridgeAR BridgeAR approved these changes

@geeksilva97 geeksilva97 geeksilva97 approved these changes

@trivikr trivikr trivikr approved these changes

@RafaelGSS RafaelGSS RafaelGSS approved these changes

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

@RaisinTen RaisinTen RaisinTen approved these changes

Assignees
No one assigned
Labels
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.