docs(governance): add an RFC process for TeachLink Web - #1591
Merged
RUKAYAT-CODER merged 1 commit intoSep 24, 2026
Merged
Conversation
|
@ameeribro4-sudo 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! 🚀 |
Contributor
|
Thank you for contributing to the project. |
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.
Summary
Closes #1494
Adds
Governance/processes/RFC_PROCESS.md, the RFC (Request for Comments) process for TeachLink Web. The document defines when a design change requires an RFC, the five lifecycle stages with owners and exit criteria for each, and explicit acceptance and rejection criteria. The single most important design decision is that the process is written to the repository as it exists today: it routes structural change through one predictable path while keeping small, bounded fixes on the normal issue/PR flow, and it records decisions with their reasoning so rejected proposals remain a durable record.Also closes
Closes #1496, Closes #1495, Closes #1493
Why
The
Governance/folder had no process defining how design proposals are raised, reviewed, and decided. Contributors could not predict whether a substantial change needed a review cycle before implementation, when decisions would be made, or by whom. The RFC process closes that gap so structural work is designed and reviewed before large effort is spent, and gives maintainers an explicit acceptance/rejection contract to apply consistently.What was built
Governance/processes/RFC_PROCESS.mdThe document follows the structure used by the existing governance policies (
Purpose→Scope→ sections →Ownership→Success) soGovernance/stays uniform.Integration changes outside
Governance/No existing files modified — the change adds one Markdown file inside
Governance/processes/only, so regression risk is low.Acceptance criteria coverage (primary issue)
Governance/processes/RFC_PROCESS.mdcreated, covering when an RFC is required, the lifecycle stages, and acceptance/rejection criteria)Governance/processes/RFC_PROCESS.md)git diff --statagainstorigin/mainshows a single file underGovernance/)Deliberately deferred
The ADR process (#1496) covers lightweight architecture decision records for decisions that do not warrant a full community RFC; keeping it separate preserves the two-level design-decision pattern and is tracked by its own issue.
Test plan
git diff origin/main --stat— exactly 1 file (Governance/processes/RFC_PROCESS.md), nothing outsideGovernance/Governance/CHARTER.md,Governance/README.md, and the hotfix process inGovernance/processes/HOTFIX.mdtype-check,lint,build,test,security-audit) on the PR.Env vars / Notes
No environment variables or configuration keys are introduced. Change confined to
Governance/.