-
-
Notifications
You must be signed in to change notification settings - Fork 142
charter: add TSC membership terms, mandatory mediation, and removal requirements #318
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
Closed
Closed
Changes from 1 commit
Commits
Show all changes
14 commits
Select commit
Hold shift + click to select a range
1e51330
charter: add TSC membership term limits and mandatory mediation
jasnell 928eb4f
[Squash] Use recertification term instead of limits
jasnell 7e61300
[Squash] Refine language around mediation
jasnell 9e8dc17
[Squash] Address nit
jasnell 86a5ca3
[Squash] Refine language around recertification and ballot confidenti…
jasnell 903b9d0
[Squash] typo
jasnell 5b6abdc
[Squash] Additional details and requirements for removal votes
jasnell 4085455
[Squash] Clarify language around mediator selection
jasnell e2c48a2
[Squash] Nits
jasnell 298ad2a
[Squash] address feedback
jasnell 01c868f
[Squash] Remove collaborator review bits
jasnell 4ba8680
[Squash] Nit
jasnell 6766814
[Squash] Address feedback
jasnell 7b6d2ae
[Squash] Updates
jasnell File filter
Filter by extension
Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
[Squash] Nits
- Loading branch information
commit e2c48a28a6172b4b28048d9b1fbdebc449eae750
There are no files selected for viewing
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
I would suggest a higher number, at least 5, given the TSC’s responsibility 3 people is to few. I would also suggest to document what happens if the number falls below the minimum.
Another topic that might fit better into a follow up pull request is to require "active" members, and define what "active" means, to avoid a situation where people are voted into the board without fulfilling their responsibilities.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
-1 on making the minimum five. What happens if the number falls below this is the responsibility of the Node.js Foundation Board to determine.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Regarding "active" members, the TSC recently passed modification to the TSC Charter that includes specific guidelines around TSC member activity. Specifically, there is a minimum participation requirement already covered by the charter.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
@gr2m FYI, this section of the charter includes these minimum requirements to stay active:
(Not saying there isn't room for improvement. Just pointing out what's already there.)