|
1 | 1 | # Maintainers
|
2 | 2 |
|
| 3 | +## Maintainer Scopes, GitHub Roles and GitHub Teams |
| 4 | + |
| 5 | +Maintainers are assigned the following scopes in this repository: |
| 6 | + |
| 7 | +| Scope | Definition | GitHub Role | GitHub Team | |
| 8 | +| ---------- | ------------------------ | ----------- | ----------------------------------- | |
| 9 | +| Admin | | Admin | [aries-admins] | |
| 10 | +| Maintainer | The GitHub Maintain role | Maintain | [aries-askar committers] | |
| 11 | +| Maintainer | The GitHub Maintain role | Maintain | [aries committers] | |
| 12 | +| Read | The GitHub Read role | Read | [Aries Contributors] | |
| 13 | +| Read | The GitHub Read role | Read | [TOC] | |
| 14 | + |
| 15 | +[aries-admins]: https://github.com/orgs/hyperledger/teams/aries-admins |
| 16 | +[aries-cloudagent-python committers]: https://github.com/orgs/hyperledger/teams/aries-cloudagent-python-committers |
| 17 | +[aries-askar committers]: https://github.com/orgs/hyperledger/teams/aries-askar-committers |
| 18 | +[Aries Contributors]: https://github.com/orgs/hyperledger/teams/aries-contributors |
| 19 | +[TOC]: https://github.com/orgs/hyperledger/teams/toc |
| 20 | + |
3 | 21 | ## Active Maintainers
|
4 | 22 |
|
5 | 23 | <!-- Please keep this sorted alphabetically by github -->
|
6 | 24 |
|
7 |
| -| Name | Github | LFID | |
8 |
| -| ---------------- | ---------------- | ---------------- | |
9 |
| -| Andrew Whitehead | andrewwhitehead | | |
10 |
| -| Berend Sliedrecht | blu3beri | | |
11 |
| -| Ian Costanzo | ianco | | |
12 |
| -| Wade Barnes | WadeBarnes | | |
| 25 | +| GitHub ID | Name | Scope | LFID | Discord ID | Email | Company Affiliation | |
| 26 | +| --------------- | ----------------- | ---------- | ---- | ---------- | ------------------------ | ------------------- | |
| 27 | +| andrewwhitehead | Andrew Whitehead | Admin | | | [email protected] | BC Gov | |
| 28 | +| dbluhm | Daniel Bluhm | Admin | | | [email protected] | Indicio PBC | |
| 29 | +| blu3beri | Berend Sliedrecht | Maintainer | | | [email protected] | Animo Solutions | |
| 30 | +| dh1128 | Daniel Hardman | Admin | | | [email protected] | Provident | |
| 31 | +| ianco | Ian Costanzo | Maintainer | | | [email protected] | Anon Solutions | |
| 32 | +| nage | Nathan George | Maintainer | | | [email protected] | Kiva | |
| 33 | +| swcurran | Stephen Curran | Admin | | | [email protected] | BC Gov | |
| 34 | +| WadeBarnes | Wade Barnes | Admin | | | [email protected] | BC Gov | |
13 | 35 |
|
14 | 36 | ## Emeritus Maintainers
|
15 | 37 |
|
16 |
| -| Name | Github | LFID | |
17 |
| -|--------------|---------|---------| |
18 |
| -| | | | |
| 38 | +| Name | GitHub ID | Scope | LFID | Discord ID | Email | Company Affiliation | |
| 39 | +|----- | --------- | ----- | ---- | ---------- | ----- | ------------------- | |
| 40 | +| | | | | | | | |
| 41 | + |
| 42 | +## The Duties of a Maintainer |
| 43 | + |
| 44 | +Maintainers are expected to perform the following duties for this repository. The duties are listed in more or less priority order: |
| 45 | + |
| 46 | +- Review, respond, and act on any security vulnerabilities reported against the repository. |
| 47 | +- Review, provide feedback on, and merge or reject GitHub Pull Requests from |
| 48 | + Contributors. |
| 49 | +- Review, triage, comment on, and close GitHub Issues |
| 50 | + submitted by Contributors. |
| 51 | +- When appropriate, lead/facilitate architectural discussions in the community. |
| 52 | +- When appropriate, lead/facilitate the creation of a product roadmap. |
| 53 | +- Create, clarify, and label issues to be worked on by Contributors. |
| 54 | +- Ensure that there is a well defined (and ideally automated) product test and |
| 55 | + release pipeline, including the publication of release artifacts. |
| 56 | +- When appropriate, execute the product release process. |
| 57 | +- Maintain the repository CONTRIBUTING.md file and getting started documents to |
| 58 | + give guidance and encouragement to those wanting to contribute to the product, and those wanting to become maintainers. |
| 59 | +- Contribute to the product via GitHub Pull Requests. |
| 60 | +- Monitor requests from the Hyperledger Technical Oversight Committee about the |
| 61 | +contents and management of Hyperledger repositories, such as branch handling, |
| 62 | +required files in repositories and so on. |
| 63 | +- Contribute to the Hyperledger Project's Quarterly Report. |
19 | 64 |
|
20 | 65 | ## Becoming a Maintainer
|
21 | 66 |
|
22 |
| -The Aries Askar community welcomes contributions. Contributors may progress to become a |
23 |
| -maintainer. To become a maintainer the following steps occur, roughly in order. |
24 |
| - |
25 |
| -- 5 significant changes have been authored by the proposed maintainer and |
26 |
| - accepted. |
27 |
| -- The proposed maintainer has the sponsorship of at least one other maintainer. |
28 |
| - - This sponsoring maintainer will create a PR modifying the list of |
29 |
| - maintainers. |
30 |
| - - The proposed maintainer accepts the nomination and expresses a willingness |
31 |
| - to be a long-term (more than 6 month) committer. |
32 |
| - - This would be a comment in the above PR. |
33 |
| - - This PR will be communicated in all appropriate communication channels. It |
34 |
| - should be mentioned in any maintainer/community call. It should also be |
35 |
| - posted to the appropriate mailing list or chat channels if they exist. |
36 |
| -- Approval by at least 3 current maintainers within two weeks of the proposal or |
37 |
| - an absolute majority of current maintainers. |
38 |
| - - These votes will be recorded in the PR modifying the list of maintainers. |
39 |
| -- No veto by another maintainer within two weeks of proposal are recorded. |
40 |
| - - All vetoes must be accompanied by a public explanation as a comment in the |
41 |
| - PR for adding this maintainer |
42 |
| - - The explanation of the veto must be reasonable. |
43 |
| - - A veto can be retracted, in that case the approval/veto timeframe is reset. |
44 |
| - - It is bad form to veto, retract, and veto again. |
45 |
| -- The proposed maintainer becomes a maintainer |
46 |
| - - Either two weeks have passed since the third approval, |
47 |
| - - Or an absolute majority of maintainers approve. |
48 |
| - - In either case, no maintainer presents a veto. |
| 67 | +This community welcomes contributions. Interested contributors are encouraged to |
| 68 | +progress to become maintainers. To become a maintainer the following steps |
| 69 | +occur, roughly in order. |
| 70 | + |
| 71 | +- The proposed maintainer establishes their reputation in the community, |
| 72 | + including authoring five (5) significant merged pull requests, and expresses |
| 73 | + an interest in becoming a maintainer for the repository. |
| 74 | +- A PR is created to update this file to add the proposed maintainer to the list of active maintainers. |
| 75 | +- The PR is authored by an existing maintainer or has a comment on the PR from an existing maintainer supporting the proposal. |
| 76 | +- 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. |
| 77 | + - The PR or comment from the proposed maintainer must include their |
| 78 | + willingness to be a long-term (more than 6 month) maintainer. |
| 79 | +- Once the PR and necessary comments have been received, an approval timeframe begins. |
| 80 | +- 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. |
| 81 | +- The PR is merged and the proposed maintainer becomes a maintainer if either: |
| 82 | + - Two weeks have passed since at least three (3) Maintainer PR approvals have been recorded, OR |
| 83 | + - An absolute majority of maintainers have approved the PR. |
| 84 | +- If the PR does not get the requisite PR approvals, it may be closed. |
| 85 | +- Once the add maintainer PR has been merged, any necessary updates to the GitHub Teams are made. |
49 | 86 |
|
50 | 87 | ## Removing Maintainers
|
51 | 88 |
|
52 |
| -Being a maintainer is not a status symbol or a title to be maintained |
| 89 | +Being a maintainer is not a status symbol or a title to be carried |
53 | 90 | indefinitely. It will occasionally be necessary and appropriate to move a
|
54 | 91 | maintainer to emeritus status. This can occur in the following situations:
|
55 | 92 |
|
56 | 93 | - Resignation of a maintainer.
|
57 | 94 | - Violation of the Code of Conduct warranting removal.
|
58 | 95 | - Inactivity.
|
59 | 96 | - A general measure of inactivity will be no commits or code review comments
|
60 |
| - for one reporting quarter, although this will not be strictly enforced if |
| 97 | + for one reporting quarter. This will not be strictly enforced if |
61 | 98 | the maintainer expresses a reasonable intent to continue contributing.
|
62 | 99 | - Reasonable exceptions to inactivity will be granted for known long term
|
63 | 100 | leave such as parental leave and medical leave.
|
64 |
| -- Other unspecified circumstances. |
| 101 | +- Other circumstances at the discretion of the other Maintainers. |
| 102 | + |
| 103 | +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 |
| 104 | +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: |
65 | 105 |
|
66 |
| -Like adding a maintainer the record and governance process for moving a |
67 |
| -maintainer to emeritus status is recorded in the github PR making that change. |
| 106 | +- A PR is created to update this file to move the maintainer to the list of emeritus maintainers. |
| 107 | +- The PR is authored by, or has a comment supporting the proposal from, an existing maintainer or Hyperledger GitHub organization administrator. |
| 108 | +- Once the PR and necessary comments have been received, the approval timeframe begins. |
| 109 | +- The PR **MAY** be communicated on appropriate communication channels, including relevant community calls, chat channels and mailing lists. |
| 110 | +- The PR is merged and the maintainer transitions to maintainer emeritus if: |
| 111 | + - The PR is approved by the maintainer to be transitioned, OR |
| 112 | + - Two weeks have passed since at least three (3) Maintainer PR approvals have been recorded, OR |
| 113 | + - An absolute majority of maintainers have approved the PR. |
| 114 | +- If the PR does not get the requisite PR approvals, it may be closed. |
68 | 115 |
|
69 | 116 | Returning to active status from emeritus status uses the same steps as adding a
|
70 | 117 | new maintainer. Note that the emeritus maintainer already has the 5 required
|
|
0 commit comments