-
-
Notifications
You must be signed in to change notification settings - Fork 1.2k
Document the governance model and the path to maintainership #8329
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
Document the governance model and the path to maintainership #8329
Conversation
439dfc9
to
b4330a7
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.
A few suggestion.
In general, I tend to prefer some ambiguity when it comes to these things. Nobody should think they are entitled to become a maintainer / or even an admin.
However, I do understand the intention to have something written and agreed upon.
I feel like some defining characteristics of pylint that are not up for discussion should be explained here as well, I'm thinking of:
I'm sure there are other implicit rules that we agree on but I don't think of anything else right now ... |
your review, you're going to be able to merge those yourself soon). | ||
- Have an admin suggest that you become maintainer, without you asking | ||
- Get unanimous approval or neutral agreement from current maintainers. |
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.
Maybe it's self explanatory so not sure it should be added:
Have been a triager for x amount of time / multiple months
If it's added, something similar could be added to the admin section as well.
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 great. Thanks for doing this @Pierre-Sassoulas |
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.
Thanks for doing this! I don't care too much about it as I think most of this is fairly self explanatory but might be good to document it.
I said this before but I do feel the OpenSSF is focusing an awful lot on minor details that take away from actually improving projects that fall under it, but oh wel...
Apply suggestions from code review Co-authored-by: Jacob Walls <jacobtylerwalls@gmail.com>
1d90028
to
d357be6
Compare
I'm not doing this only for OpenSSF. There are more fundamental requirements of OpenSSF, but we're already very advanced on what we do regarding fundamental stuff (license/tests/bug reporting/ documentation/CI/rights), so the actual way to make pylint better are becoming more and more "niche". Being explicit about the code of conduct being enforced for everyone is important (to me) due to PyCQA meta issue 54. I'm open on working for specific aspects that you find more impactful next, see https://bestpractices.coreinfrastructure.org/en/projects/6328 |
Thank you for the review @mbyrnepr2 :) |
Type of Changes
Description
This is done due to clarify our standing on moderation issues related to the recent PyCQA discussion, and also as part of the work done for the OpenSSF best practices (that Tidelift is paying me to do to make the software supply chain more secure). To be clear having an autocrat behaving like an asshole as the owner of a project is a liability for everyone in the software supply chain. Making sure that this does not happen for pylint is valuable. The way to become a maintainer is also a frequently asked question.