-
Notifications
You must be signed in to change notification settings - Fork 201
Fix/update governance two-thirds super-majority link #411
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
Open
gjenkins8
wants to merge
1
commit into
helm:main
Choose a base branch
from
gjenkins8:patch-3
base: main
Could not load branches
Branch not found: {{ refName }}
Loading
Could not load tags
Nothing to show
Loading
Are you sure you want to change the base?
Some commits from the old base branch may be removed from the timeline,
and old review comments may become outdated.
Open
Changes from all commits
Commits
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
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
Original file line number | Diff line number | Diff line change | ||||
---|---|---|---|---|---|---|
|
@@ -43,7 +43,7 @@ Changes to org maintainers use the following: | |||||
* Any project maintainer of any active (non-archived) Helm organization project is eligible for a position as an org maintainer. | ||||||
* No one company or organization can employ a simple majority of the org maintainers | ||||||
* An org maintainer may step down by emailing the org maintainers mailing list. Within 7 calendar days the [helm mailing list](https://lists.cncf.io/g/cncf-helm) needs to be notified of the change | ||||||
* Org maintainers MUST remain active on the project. If they are unresponsive for > 3 months they will lose org maintainership unless a [super-majority](https://en.wikipedia.org/wiki/Supermajority#Two-thirds_vote) of the other org maintainers agrees to extend the period to be greater than 3 months | ||||||
* Org maintainers MUST remain active on the project. If they are unresponsive for > 3 months they will lose org maintainership unless a [two-thirds super-majority] of the other org maintainers agrees to extend the period to be greater than 3 months | ||||||
* When there is an opening for a new org maintainer, any person who has made a contribution to any repo under the Helm GitHub org may nominate a suitable project maintainer of an active project as a replacement | ||||||
* The nomination period will be three weeks starting the day after an org maintainer opening becomes available | ||||||
* The nomination must be made via the [public Helm mailing list](https://lists.cncf.io/g/cncf-helm/) | ||||||
|
@@ -71,8 +71,8 @@ Project maintainers do not need to be software developers. No explicit role is p | |||||
Changes to maintainers use the following: | ||||||
|
||||||
* A maintainer may step down by emailing the [Helm mailing list](https://lists.cncf.io/g/cncf-helm) | ||||||
* Maintainers MUST remain active. If they are unresponsive for > 3 months they will be automatically removed unless a [super-majority](https://en.wikipedia.org/wiki/Supermajority#Two-thirds_vote) of the other project maintainers agrees to extend the period to be greater than 3 months | ||||||
* New maintainers can be added to a project by a [super-majority](https://en.wikipedia.org/wiki/Supermajority#Two-thirds_vote) vote of the existing maintainers. While nomination will happen on the public [Helm mailing list](https://lists.cncf.io/g/cncf-helm), voting will happen on the private maintainer list. | ||||||
* Maintainers MUST remain active. If they are unresponsive for > 3 months they will be automatically removed unless a [two-thirds super-majority] of the other project maintainers agrees to extend the period to be greater than 3 months | ||||||
There was a problem hiding this comment. Choose a reason for hiding this commentThe reason will be displayed to describe this comment to others. Learn more.
Suggested change
|
||||||
* New maintainers can be added to a project by a [two-thirds super-majority] vote of the existing maintainers. While nomination will happen on the public [Helm mailing list](https://lists.cncf.io/g/cncf-helm), voting will happen on the private maintainer list. | ||||||
There was a problem hiding this comment. Choose a reason for hiding this commentThe reason will be displayed to describe this comment to others. Learn more.
Suggested change
|
||||||
* When a project has no maintainers the Helm org maintainers become responsible for it and may archive the project or find new maintainers | ||||||
* When maintainers no longer maintain active projects due to project archival or removal, they become emeritus maintainers unless they join another Helm project as maintainers. | ||||||
|
||||||
|
@@ -108,3 +108,5 @@ The following licenses and contributor agreements will be used for Helm projects | |||||
* [Apache 2.0](https://opensource.org/licenses/Apache-2.0) for code | ||||||
* [Creative Commons Attribution 4.0 International Public License](https://creativecommons.org/licenses/by/4.0/legalcode) for documentation | ||||||
* [Developer Certificate of Origin](https://developercertificate.org/) for new contributions | ||||||
|
||||||
[two-thirds super-majority]: https://en.wikipedia.org/wiki/Supermajority#Two-thirds |
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.
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.
this is a reference style link
source for the above: