Add explicit collaborator list into README.md as per GOVERNANCE.md#2455
Add explicit collaborator list into README.md as per GOVERNANCE.md#2455sxa wants to merge 2 commits intonodejs:mainfrom
Conversation
This comment was marked as resolved.
This comment was marked as resolved.
|
It would probably make sense to add a workflow similar to https://github.com/nodejs/node/blob/HEAD/tools/lint-readme-lists.mjs / https://github.com/nodejs/node/blob/dd3fe4997bc2a50cfb06282ba53d3af6877c907e/.github/workflows/linters.yml#L257-L286 to help keep the README in sync with the actual team |
Signed-off-by: Stewart X Addison <sxa@ibm.com>
|
Above force was to rebase on top fo the linker fixes.
Yep, we could look at that later but for the purposes of this PR (Which is ultimately to assist with giving Mike collaborator access in the repository) let's not expand the scope. The issue we've had with this repository is not having many people active to be able to do fixes etc. |
README.md
Outdated
| Additional collaborators are managed via the | ||
| [@nodejs/docker team](https://github.com/orgs/nodejs/teams/docker). |
There was a problem hiding this comment.
I suggest to have the list of names appear before the comment about how to add names.
There was a problem hiding this comment.
I don't mind it being before, but I've made that change and also removed "Additional" since "managed" implies addition and removal.
|
According to the GOVERNANCE document, it is the responsibility of Maintainers to manage and approve membership, so at least one of the following would need to approve this PR actively:
Since the list of Collaborators is now being published, it would also make sense for Collaborators to confirm their agreement:
|
Signed-off-by: Stewart X Addison <sxa@ibm.com>
MikeMcC399
left a comment
There was a problem hiding this comment.
Thanks for responding to comments!
Adding copy of the collaborator list into here in order to allow Pull Requests to be used for the process as per the governance doc
Ref: discussion in #2448
Description
Motivation and Context
Testing Details
Example Output(if appropriate)
Types of changes
Checklist