Conversation
Adds Governance/processes/EMERITUS_TRANSITION.md defining when a member moves to emeritus, the privileges retained and withdrawn, and the path back to active status, with Governance/processes/EMERITUS_TRANSITION.test.ts pinning the guarantees to the document.
|
@cythecode Great news! 🎉 Based on an automated assessment of this PR, the linked Wave issue(s) no longer count against your application limits. You can now already apply to more issues while waiting for a review of this PR. Keep up the great work! 🚀 |
There was a problem hiding this comment.
Copilot review overview
🟡 Changes recommended
Unresolved moderate documentation gaps and a critical test-runtime failure remain.
Review effort: Lite
Findings: 1
Open (2)
What changed in this PR
Adds an emeritus transition process and regression tests under Governance/.
Changes:
- Documents transition triggers, deadlines, privileges, and return-to-active paths.
- Adds tests validating the process structure and governance references.
| File | Summary and findings |
|---|---|
Governance/processes/EMERITUS_TRANSITION.md |
Documents the emeritus lifecycle. Five moderate findings remain: clarify contributor return timing, handle the absent CODEOWNERS file, define return paths for all scoped roles, assign the initial recording step, and explicitly revoke private-channel and external-representation privileges. |
Governance/processes/EMERITUS_TRANSITION.test.ts |
Adds regression tests. One critical finding remains: replace __dirname with an ES-module-compatible path derived from import.meta.url at the noted locations. |
💡 Add a code-review agent skill or configure MCP servers for context-aware, tailored reviews. Learn more in the docs.
Comment on lines
+15
to
+17
| import path from 'node:path'; | ||
|
|
||
| const PROCESS_PATH = path.resolve(__dirname, 'EMERITUS_TRANSITION.md'); |
Comment on lines
+109
to
+115
| - **Reviewer or moderator.** The member demonstrates re-engagement through at | ||
| least two completed reviews or moderation actions, and one active | ||
| maintainer records approval on the return issue. | ||
| - **Maintainer.** The member demonstrates re-engagement through at least four | ||
| completed reviews or merged pull requests within 60 days, and two active | ||
| maintainers who are not the requestor record approval, followed by the | ||
| onboarding steps in `Governance/roles/MAINTAINER.md`. |
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.


Overview
Adds
Governance/processes/EMERITUS_TRANSITION.md: the operational companion toGovernance/roles/EMERITUS.md. The role document defines what emeritus status is;this process defines the transition itself — when a member moves to emeritus, which
privileges are retained and which are withdrawn, and the recorded path back to active
status, each step with an owner and a deadline.
The process names three triggers (voluntary step-down, end of a fixed term, role
consolidation), runs all of them through one recorded sequence (request or consented
proposal, acknowledgement within 3 business days, recorded decision within 10 business
days, privileges applied within 5 business days), keeps emeritus opt-in and separate
from the inactivity and misconduct paths, and matches the reinstatement thresholds
already committed in
Governance/policies/INACTIVITY.md.Related Issue
Fixes the governance gap tracked in #1488:
Governance/had noGovernance/processes/EMERITUS_TRANSITION.md, so the transition into emeritus and thepath back to active status were not written down in a single, versioned reference.
Changes
Emeritus transition process
[ADD]
Governance/processes/EMERITUS_TRANSITION.mdsequence they all run, with the acknowledgement, decision, and apply deadlines, and
the rule that a proposal without the member's consent does not proceed.
participation, a non-voting voice at meetings, and consented informal consultation.
repository permissions, CODEOWNERS placement, private-channel access, and external
representation.
reviewer/moderator after two completed reviews with one maintainer approval,
maintainer after four reviews or pulls in 60 days with two approvals and onboarding,
plus appointment by nomination under
Governance/processes/NOMINATION.md, and a3-business-day acknowledgement / 10-business-day decision clock.
Governance/roles/EMERITUS.md,Governance/roles/CONTRIBUTOR.md,Governance/roles/MAINTAINER.md,Governance/policies/INACTIVITY.md,Governance/RECOGNITION.md, andGovernance/CODE_OF_CONDUCT.md.[ADD]
Governance/processes/EMERITUS_TRANSITION.test.tswithdrawn privileges, and the return path to the checked-in document, and checking
the thresholds against
Governance/policies/INACTIVITY.mdandGovernance/roles/EMERITUS.mdat test time.Verification Results
Governance/processes/EMERITUS_TRANSITION.mddefines the transition, retained/withdrawn privileges, and return pathGovernance/processes/Closes #1488