Skip to content

Commit 773c951

Browse files
authored
Merge pull request #1706 from hyperledger/swcurran-patch-1
Update MAINTAINERS.md to reference access control file
2 parents 548e0ee + 4c295dd commit 773c951

File tree

1 file changed

+31
-65
lines changed

1 file changed

+31
-65
lines changed

MAINTAINERS.md

Lines changed: 31 additions & 65 deletions
Original file line numberDiff line numberDiff line change
@@ -2,49 +2,14 @@
22

33
## Maintainer Scopes, GitHub Roles and GitHub Teams
44

5-
Maintainers are assigned the following scopes in this repository:
6-
7-
| Scope | Definition | GitHub Role | GitHub Team |
8-
| ---------- | ------------------------ | ----------- | ----------------------------------- |
9-
| Admin | | Admin | [indy-admins] |
10-
| Maintainer | The GitHub Maintain role | Maintain | [indy-plenum-maintainers] |
11-
| Maintainer | The GitHub Maintain role | Maintain | [Indy Common] |
12-
| Write | The GitHub Write role | Write | [Indy CI] |
13-
14-
[indy-admins]: https://github.com/orgs/hyperledger/teams/indy-admins
15-
[indy-plenum-maintainers]: https://github.com/orgs/hyperledger/teams/indy-plenum-maintainers
16-
[Indy CI]: https://github.com/orgs/hyperledger/teams/indy-ci
17-
[Indy Common]: https://github.com/orgs/hyperledger/teams/indy-common
18-
19-
## Active Maintainers
20-
21-
<!-- Please keep this sorted alphabetically by github -->
22-
23-
| GitHub ID | Name | Scope | LFID | Discord ID | Email | Company Affiliation |
24-
| ------------------ | ----------------- | ---------- | ---- | ---------- | ------------------------------ | ------------------------- |
25-
| andrewwhitehead | Andrew Whitehead | Admin | | | cywolf@gmail.com | BC Gov |
26-
| anikitinDSR | Andrew Nikitin | Maintainer | | | | |
27-
| askolesov | Alexander Kolesov | Maintainer | | | | |
28-
| brentzundel | Brent Zundel | Maintainer | | | Brent.Zundel@gendigital.com | Gen Digital |
29-
| dhh1128 | Daniel Hardman | Admin | | | daniel.hardman@gmail.com | Provident |
30-
| esplinr | Richard Esplin | Admin | | | | |
31-
| hyperledger-ci | Hyperledger | | | | rjones@linuxfoundation.org | |
32-
| hyperledger-github | Hyperledger | Admin | | | rjones@linuxfoundation.org | |
33-
| mikelodder7 | Michael Lodder | Maintainer | | | | |
34-
| nage | Nathan George | Admin | | | nathang@kiva.org | Kiva |
35-
| pSchlarb | Philipp Schlarb | Maintainer | | | p.schlarb@esatus.com | esatus |
36-
| SovBot | Sovrin Bot | Write | | | support@sovrin.org | Sovrin Foundation |
37-
| swcurran | Stephen Curran | Admin | | | swcurran@cloudcompass.ca | BC Gov |
38-
| toktar | Renata Toktar | Maintainer | | | | |
39-
| udosson | Robin Klemens | Maintainer | | | klemens@internet-sicherheit.de | |
40-
| vimmerru | Vyacheslav Gudkov | Maintainer | | | | |
41-
| WadeBarnes | Wade Barnes | Admin | | | wade@neoterictech.ca | BC Gov, Sovrin Foundation |
42-
43-
## Emeritus Maintainers
44-
45-
| Name | GitHub ID | Scope | LFID | Discord ID | Email | Company Affiliation |
46-
|----- | --------- | ----- | ---- | ---------- | ----- | ------------------- |
47-
| | | | | | | |
5+
The Maintainers of this repo, defined as GitHub users with escalated privileges
6+
in the repo, are managed in the Hyperledger "governance" repo's [access control file](https://github.com/hyperledger/governance/blob/main/access-control.yaml). Consult that to see:
7+
8+
- What teams have escalated privileges to this repository.
9+
- What GitHub roles those teams have in the repository.
10+
- Who are the members of each of those teams.
11+
12+
The actions covered below for [becoming](#becoming-a-maintainer) and [removing](#removing-maintainers) are made manifest through PRs to that file.
4813

4914
## The Duties of a Maintainer
5015

@@ -78,18 +43,18 @@ occur, roughly in order.
7843
- The proposed maintainer establishes their reputation in the community,
7944
including authoring five (5) significant merged pull requests, and expresses
8045
an interest in becoming a maintainer for the repository.
81-
- A PR is created to update this file to add the proposed maintainer to the list of active maintainers.
82-
- The PR is authored by an existing maintainer or has a comment on the PR from an existing maintainer supporting the proposal.
83-
- The PR is authored by the proposed maintainer or has a comment on the PR from the proposed maintainer confirming their interest in being a maintainer.
84-
- The PR or comment from the proposed maintainer must include their
46+
- An issue is created to add the proposed maintainer to the list of active maintainers.
47+
- The issue is authored by an existing maintainer or has a comment on the PR from an existing maintainer supporting the proposal.
48+
- The issue is authored by the proposed maintainer or has a comment on the issue from the proposed maintainer confirming their interest in being a maintainer.
49+
- The issue or comment from the proposed maintainer must include their
8550
willingness to be a long-term (more than 6 month) maintainer.
86-
- Once the PR and necessary comments have been received, an approval timeframe begins.
87-
- The PR **MUST** be communicated on all appropriate communication channels, including relevant community calls, chat channels and mailing lists. Comments of support from the community are welcome.
88-
- The PR is merged and the proposed maintainer becomes a maintainer if either:
89-
- Two weeks have passed since at least three (3) Maintainer PR approvals have been recorded, OR
90-
- An absolute majority of maintainers have approved the PR.
91-
- If the PR does not get the requisite PR approvals, it may be closed.
92-
- Once the add maintainer PR has been merged, any necessary updates to the GitHub Teams are made.
51+
- Once the issue and necessary comments have been received, an approval timeframe begins.
52+
- The issue **MUST** be communicated on all appropriate communication channels, including relevant community calls, chat channels and mailing lists. Comments of support from the community are welcome.
53+
- The issue is approved and the proposed maintainer becomes a maintainer if either:
54+
- Two weeks have passed since at least three (3) Maintainer issue approvals have been recorded, OR
55+
- An absolute majority of maintainers have approved the issue.
56+
- If the issue does not get the requisite approvals, it may be closed.
57+
- Once the add maintainer issue has been approved, the necessary updates to the GitHub Teams are made via a PR to the Hyperledger "governance" repo's [access control file](https://github.com/hyperledger/governance/blob/main/access-control.yaml).
9358

9459
## Removing Maintainers
9560

@@ -108,17 +73,18 @@ maintainer to emeritus status. This can occur in the following situations:
10873
- Other circumstances at the discretion of the other Maintainers.
10974

11075
The process to move a maintainer from active to emeritus status is comparable to the process for adding a maintainer, outlined above. In the case of voluntary
111-
resignation, the Pull Request can be merged following a maintainer PR approval. If the removal is for any other reason, the following steps **SHOULD** be followed:
112-
113-
- A PR is created to update this file to move the maintainer to the list of emeritus maintainers.
114-
- The PR is authored by, or has a comment supporting the proposal from, an existing maintainer or Hyperledger GitHub organization administrator.
115-
- Once the PR and necessary comments have been received, the approval timeframe begins.
116-
- The PR **MAY** be communicated on appropriate communication channels, including relevant community calls, chat channels and mailing lists.
117-
- The PR is merged and the maintainer transitions to maintainer emeritus if:
118-
- The PR is approved by the maintainer to be transitioned, OR
119-
- Two weeks have passed since at least three (3) Maintainer PR approvals have been recorded, OR
120-
- An absolute majority of maintainers have approved the PR.
121-
- If the PR does not get the requisite PR approvals, it may be closed.
76+
resignation, the Pull Request can be merged following a maintainer issue approval. If the removal is for any other reason, the following steps **SHOULD** be followed:
77+
78+
- An issue is created to move the maintainer to the list of emeritus maintainers.
79+
- The issue is authored by, or has a comment supporting the proposal from, an existing maintainer or Hyperledger GitHub organization administrator.
80+
- Once the issue and necessary comments have been received, the approval timeframe begins.
81+
- The issue **MAY** be communicated on appropriate communication channels, including relevant community calls, chat channels and mailing lists.
82+
- The issue is approved and the maintainer transitions to maintainer emeritus if:
83+
- The issue is approved by the maintainer to be transitioned, OR
84+
- Two weeks have passed since at least three (3) Maintainer issue approvals have been recorded, OR
85+
- An absolute majority of maintainers have approved the issue.
86+
- If the issue does not get the requisite approvals, it may be closed.
87+
- Once the remove maintainer issue has been approved, the necessary updates to the GitHub Teams are made via a PR to the Hyperledger "governance" repo's [access control file](https://github.com/hyperledger/governance/blob/main/access-control.yaml).
12288

12389
Returning to active status from emeritus status uses the same steps as adding a
12490
new maintainer. Note that the emeritus maintainer already has the 5 required

0 commit comments

Comments
 (0)