chore/update community governance#8
Open
mingcheng wants to merge 1 commit into
Open
Conversation
Contributor
Author
|
This is a BIG PR, please review before merge! |
There was a problem hiding this comment.
Pull request overview
This PR modernizes the AReaL community governance documentation by introducing a structured contributor ladder and membership/election processes, splitting role holder lists into dedicated roles/ documents, and expanding contributor guidance. It also updates the weekly pulse workflow to generate and open PRs automatically with the generated report.
Changes:
- Reworks governance docs to define principles, decision-making, roles ladder, and formal membership/election procedures.
- Adds
roles/directory docs for Maintainers/Approvers/Members/Security Team and updates references from existing docs. - Enhances automation: weekly pulse workflow now writes reports to
reports/and opens a PR automatically; adds related.gitignoreentries.
Reviewed changes
Copilot reviewed 14 out of 15 changed files in this pull request and generated 3 comments.
Show a summary per file
| File | Description |
|---|---|
| SECURITY.md | Updates private reporting guidance and points to the Security Team + Maintainers role docs. |
| roles/Security-Team.md | Adds Security Team definition, composition, responsibilities, and links back to SECURITY.md. |
| roles/README.md | Adds an index README for community role holder lists under roles/. |
| roles/Members.md | Adds a Members roster document (currently empty with guidance). |
| roles/Maintainers.md | Adds a Maintainers roster document listing current maintainers. |
| roles/Approvers.md | Adds an Approvers roster document (currently empty with guidance). |
| README.md | Refreshes “What lives here” and adds community communication + roles overview. |
| GOVERNANCE.md | Major rewrite: principles, scope, decision-making, PR policy, comms, legal/compliance, amendments. |
| CONTRIBUTING.md | Adds DCO/sign-off and licensing guidance and updates quick-start commit example. |
| COMMUNITY_MEMBERSHIP.md | Introduces election/voting/membership processes, inactivity rules, and worked example. |
| COMMUNITY_LADDER.md | Introduces the Contributor→Member→Approver→Maintainer ladder with responsibilities/qualifications/privileges. |
| .gitignore | Ignores /.cache and /pulse-report.md. |
| .github/workflows/pulse.yml | Adds timestamping, report file creation, and automatic PR creation for weekly pulse reports. |
| .github/ISSUE_TEMPLATE/nominate.md | Adds a nomination issue template aligned with the new membership/election process. |
| .github/ISSUE_TEMPLATE/departure.md | Adds a departure notice issue template aligned with the new membership process. |
💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.
Comment on lines
+12
to
+14
| - **Openness.** All of our communication and decision-making happens in the open. | ||
| Discussions, proposals, and decisions are recorded in public issues, pull requests, | ||
| and community meetings so anyone can follow along. |
Comment on lines
+102
to
+105
| - All pull requests require approval from at least **two Maintainers (or Approvers in | ||
| their area of ownership)** before they can be merged. At least one approval must come | ||
| from a code owner of the modified paths (see | ||
| [`.github/CODEOWNERS`](https://github.com/areal-project/AReaL/blob/main/.github/CODEOWNERS)). |
Comment on lines
+41
to
+43
| - name: Get current timestamp | ||
| uses: mingcheng/get-timestamp-action@v1.1.0 | ||
| id: timestamp |
- create COMMUNITY_LADDER.md defining four-tier contributor roles with responsibilities and qualifications - create COMMUNITY_MEMBERSHIP.md documenting election processes, voting thresholds, and membership management - add issue templates for role nominations and departure notices in .github/ISSUE_TEMPLATE/ - reorganize GOVERNANCE.md to align with new contributor ladder and add detailed decision-making processes - create roles/ directory with separate files for Maintainers, Approvers, Members, and Security Team - update CONTRIBUTING.md with DCO sign-off requirements and AI-assisted coding guidelines - update README.md and SECURITY.md with references to new community structure documents Signed-off-by: mingcheng <mingcheng@apache.org>
mingcheng
force-pushed
the
chore/update-community-governance
branch
2 times, most recently
from
b57d2f9 to
fabab31
Compare
Comment on lines
+8
to
+10
| The Security Team consists of all current active Maintainers as listed in | ||
| [Maintainers.md](Maintainers.md). All Maintainers are members of the Security Team by | ||
| virtue of their role. |
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
No description provided.