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.

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

Add requirement that non-collaborator members be approved by the TSC #3956

Merged
merged 2 commits into from
Nov 21, 2024

Conversation

mcollina
Copy link
Member

To avoid XY-style attacks, build-wg members should be highly trusted. Therefore, if they are not already Node.js collaborators, they should be approved by the TSC.

To avoid XY-style attacks, build-wg members should be highly trusted. Therefore, if they are not already Node.js collaborators, they should be approved by the TSC.
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.

LGTM

The numbering just seems to have changed unintentionally.

GOVERNANCE.md Outdated Show resolved Hide resolved
Co-authored-by: Ruben Bridgewater <[email protected]>
@mhdawson
Copy link
Member

@mcollina does this requirement already exist for the release WG?

@mcollina
Copy link
Member Author

yes it does

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

@mhdawson
Copy link
Member

@targos any concerns with this?

@mhdawson
Copy link
Member

I think this has been open long enough and we have a number of +1s so landing.

@mhdawson mhdawson merged commit 1ebc9c1 into main Nov 21, 2024
2 checks passed
@richardlau richardlau deleted the mcollina-patch-1 branch November 21, 2024 20:45
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

9 participants