-
Notifications
You must be signed in to change notification settings - Fork 149
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 Emeritus Maintaner role; Add 2 days period for merging PRs #841
Conversation
Signed-off-by: Ricardo Zanini <[email protected]>
Signed-off-by: Ricardo Zanini <[email protected]>
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.
Overall LGTM
Just a few minor wording suggestions - even if not made, I'm ok with it.
Co-authored-by: Doug Davis <[email protected]>
Signed-off-by: Ricardo Zanini <[email protected]>
7c16bf1
to
a23ff70
Compare
@duglin I've incorporated your comments. Mind doing a quick look? |
Signed-off-by: Ricardo Zanini <[email protected]>
2d04c03
to
fd05cc7
Compare
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.
Excellent! Thanks @ricardozanini !
Many thanks for submitting your Pull Request ❤️!
Please specify parts of this PR update:
Discussion or Issue link:
Discussion took place between the Serverless WG Maintainers.
What this PR does / why we need it:
In this PR, we update our governance model to include a new
Emeritus Maintainer
role to praise maintainers who wish to move forward as advisors without the obligations of active maintainers.Additionally, we rephrased the Lazy Consensus rule to highlight that after a PR gets approved by another maintainer, others have 2 days notice before it gets merged.
Special notes for reviewers:
Additional information: