rfc43: update roles section #485
Conversation
|
|
Problem: the governance doc discussed "the project", which could be confusing since "the project" is really a github organization that contains several github projects, that may have different maintainers. Add a sentence to the Project Governance intro to make this explicit.
|
|
grondo
left a comment
grondo
left a comment
There was a problem hiding this comment.
This looks good to me. One suggestion and a couple questions inline.
spec_48.rst
Outdated
| the flux-framework organization: | ||
|
|
||
| Administrator | ||
| A person with the GitHub administrator access to the organization and all its |
There was a problem hiding this comment.
| A person with the GitHub administrator access to the organization and all its | |
| A person with GitHub administrator access to the organization and all its |
spec_48.rst
Outdated
| Contributor | ||
| A person who wishes to provide a patch. This person may be added to the | ||
| GitHub project without commit access, or they may be "free range" | ||
| contributors without special access. |
There was a problem hiding this comment.
Should Collaborator also be defined here (or actually below)? (It is used later in the document) I think there was already a discussion around the term "collaborator" but I don't remember the resolution.
I'm actually not sure what "free range" contributor means here, does that just mean they are contributing issues or pull requests from their own forks without special access? (maybe just say that?)
There was a problem hiding this comment.
Yeah, sorry, "free range" was a bit silly.
I think Collaborator was just another name for contributor. Will fix.
Problem: maintainers are mentioned outside the Roles section, and administrators are not defined. Since we are distinguishing between individual project Maintainers and the framework Administrators, be explicit about that in the Roles section.
Problem: collaborator and contributor are used interchangeably in this RFC. Use contributor.
|
Fixed those things and will set MWP. |
Merge Queue Status✅ The pull request has been merged at 5034048 This pull request spent 6 seconds in the queue, with no time running CI. Required conditions to merge
|
|
|
2 participants
@grondo, @trws, and I had a chat today about this and wanted to add Administrator as a new role in the gov doc. This would be the people who are able to add/remove projects from the flux-framework organization and do other "root" like tasks within it.
This defines the new role and tries to distinguish between "the project" which is really the organization, and the individual projects within it which follow the rules, but may have different maintainers for example. (They also may have different policies for development, within the constraints of RFC 1).
Anyway, @trws or @grondo please make sure I captured what we discussed.
p.s. I also added a README.md for the organization that lists me, @grondo, and @trws as the administrators and this doc links to that page.