Add documentation of the Decision Committee process#1030
Add documentation of the Decision Committee process#1030
Conversation
|
I will automatically update this comment whenever this PR is modified |
|
I love using GHA to create the PR from a label or manual trigger! Here's a workflow I use for creating meeting notes documents using a Jinja template. Something similar could be used to pre-populate a decision template from an issue. |
|
Also, I think the template could be the place where we keep the source of truth for the process. Then every time a PR is opened from that template, the process is right there for everyone to be on the same page about endorsement threshold, etc. And as the process changes, we have a record of which version of the process was used for each decision. |
|
I don't know what GHAs are capable of, but it would also be nice to have one comment with the kind of table @jhkennedy used to remind everyone of the approvals before they get marked as stale. |
|
💯 Perhaps that table could be pre-populated from CODEOWNERS in the PR description? |
| - Decision committee members endorse a decision as written by approving the PR | ||
| - Decision committee members can reject a decision _as written_ by requesting changes in the PR | ||
| - Decision committee member can express neutral discussion points by leaving PR comments | ||
| 5. Once N decision committee members endorse a decision (approve the PR), the decision is considered "decided" and the PR should be merged to record the decisions |
There was a problem hiding this comment.
When we can return to this, I think it will be important to remember that we created ambiguity in #1047 between "voting" and "approving". We had the good idea of creating a table to track preferences, but it was unclear (at least to me) whether that was a "straw poll" or the vote of record. We merged the PR with a single "approval", so in effect the table was the official record. Or we went with N==1, which seems insufficient. Let's ensure the process eliminates ambiguity here.
There was a problem hiding this comment.
Great call, I totally forgot about this 😆 I was thinking of the table as the vote of record, which I shouldn't have.

In the May 13th hackday, we discussed the decision committee a bit and started to develop what the actual process could/should look like.
We decided to try and use standard GitHub PR workflows to facilitate a decision, which I've outlined here, and use #1011 to work through the processes and try it out.
I'm leaving this as draft while we work through the process in #1011 as I expect we will update it as we go, but this will at least provide a frame of reference.
fixes #761
TODOs:
Note
This would be a lot easier to manage using teams, which would also be a lot easier to manage in an org, so we might want to make a decision for #929 as well
📚 Documentation preview 📚: https://earthaccess--1030.org.readthedocs.build/en/1030/