Skip to content

Commit 8838b4d

Browse files
docs(governance): list the maintainers who can actually approve a change
MAINTAINERS.md named one maintainer, the Project Lead. The same file says changes require approval by "an existing maintainer", and that a maintainer appointment needs "a 2/3 vote of the current maintainers". With one listed maintainer and a project rule that nobody self-approves, every pull request here structurally had no legitimate approver. Six accounts hold merge rights on the org and none but the Project Lead appeared in the file, so the documented process and the real one had drifted apart. Rishabh Poddar and Aaron Fulkerson are added. Neither is a new public assignment: both are already listed as Spec Editors in agentrust-io/trace-spec MAINTAINERS.md, both are at OPAQUE Systems, and both already hold merge rights here. This makes the file describe what is already true. Deliberately not added: the remaining org admins, including an external contributor at another company. Naming someone a maintainer of a public project is a real assignment, theirs to accept rather than ours to record. Also carries the Project Lead authority line from trace-spec so the three repositories describe the same structure.
1 parent 663c24e commit 8838b4d

1 file changed

Lines changed: 5 additions & 0 deletions

File tree

‎MAINTAINERS.md‎

Lines changed: 5 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -5,6 +5,11 @@
55
| Name | GitHub | Affiliation | Role |
66
|------|--------|-------------|------|
77
| Imran Siddique | @imran-siddique | OPAQUE Systems | Project Lead |
8+
| Rishabh Poddar | @podcastinator | OPAQUE Systems | Maintainer |
9+
| Aaron Fulkerson | @AaronRoeF | OPAQUE Systems | Maintainer |
10+
11+
The Project Lead has final decision authority on architecture, conformance
12+
requirements, and Maintainer appointments.
813

914
## Roles
1015

0 commit comments

Comments
 (0)