-
Notifications
You must be signed in to change notification settings - Fork 9
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
New Governance Models for Different Team Sizes #139
Conversation
NOTE: I'd like to propose (after merging this) to apply the new "Medium Size Team" template (sample here) to the SLIM Governance Model, which would mean combining the SLIM TSC and PSC. |
Some feedback received so far:
|
I feel we should do large and small and skip over medium. Someone can integrate parts of large if they want to extend the small group model. On that note, I think all SLIM example repos should default to the small model. |
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.
Approved with the proviso that we should limit the models to two -- medium (renamed small) and large.
Consider adding a diagram of a suggest path to merit. Apache Software Foundation has such an example https://training.apache.org/presentations/comdev/apache-intro/#/path-of-merit. Another consideration is at times for medium or large teams a project could chose a governance where the path to merit is sped up by having Committers automatically put onto the Steering Committee. This is just an option and a decision to be made by those that adopt one of these governance models. |
|
||
Subset of contributors who have been given write access to one or more project repositories. Both contributors and collaborators can propose changes to the project via pull requests, but only collaborators can formally review and approve (merge) these requests. Any contributor who has made a non-trivial contribution should be on-boarded as a collaborator in a timely manner. | ||
Subset of contributors who have been given write access to one or more project repositories. Both contributors and committer can propose changes to the project via pull requests, but only committers can formally review and approve (merge) these requests. Any contributor who has made a non-trivial contribution should be on-boarded as a committer in a timely manner. |
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.
A comment that I made internally (see Slack thread) about the following:
Any contributor who has made a non-trivial contribution should be on-boarded as a committer in a timely manner.
Some open-source projects might be very touchy about allowing write permissions to external contributors. Branch protection and permissions can be set in place to alleviate some concerns, but there's always a non-zero risk. For example, through allowing arbitrary commits on the main repo's branches, one could be concerned about this indirectly allowing external contributors to execute arbitrary code on self-hosted GitHub Action runners, if the repo has any.
@riverma suggested this new phrasing, which I personally really like:
Any contributor who has made a non-trivial contribution should be considered as a candidate for the committer role by the steering committee or the product manager.
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.
Thank you for your review @thomas-bc and your suggestions! Will incorporate the new phrasing 👍
💯 that @PaulMRamirez - great suggestion for adding a visual. I'll incorporate that. |
- MDX plugin to show snippets of code from external files
Re-opening this PR as work has not finished. |
Closing in place of: #170 |
Purpose
Proposed Changes
Issues
Testing