[groq] [QA][POLICY] Branch Protection + Mandatory Review Policy for All Repos (#918) #975
27
CODEOWNERS
27
CODEOWNERS
@@ -39,7 +39,32 @@ the-nexus/ai/ @Timmy
|
||||
CONTRIBUTING.md
|
||||
```diff
|
||||
<<<<<<< search
|
||||
# CODEOWNERS file for repository review requirements
|
||||
# Contribution Policy for The Nexus Organization
|
||||
|
||||
## Branch Protection Rules
|
||||
|
||||
All repositories must enforce:
|
||||
- Require Pull Request for merge
|
||||
- 1+ approvals required
|
||||
- Dismiss stale approvals
|
||||
- Require CI to pass (where CI exists)
|
||||
- Block force pushes
|
||||
- Block branch deletion
|
||||
|
||||
## Review Requirements
|
||||
|
||||
- @perplexity - Default reviewer for all repositories (QA gate)
|
||||
- @Timmy - Required reviewer for hermes-agent (owner gate)
|
||||
- Repo-specific owners for specialized areas
|
||||
|
||||
For full reviewer assignments, see [CODEOWNERS](CODEOWNERS)
|
||||
|
||||
## CI Status
|
||||
|
||||
- ✅ hermes-agent (CI active)
|
||||
- ⚠ the-nexus (CI pending #915)
|
||||
- ❌ timmy-home (no CI)
|
||||
- ❌ timmy-config (no CI)
|
||||
|
||||
# Default reviewer for all repositories
|
||||
* @perplexity
|
||||
|
||||
29
README.md
29
README.md
@@ -1,11 +1,30 @@
|
||||
# Nexus Repository Policy Documentation
|
||||
# Nexus Organization Policy
|
||||
|
||||
## Branch Protection Rules
|
||||
## Branch Protection & Review Requirements
|
||||
|
||||
All repositories must enforce the following branch protection rules on the `main` branch:
|
||||
All repositories must enforce these rules on the `main` branch:
|
||||
- ✅ Require Pull Request for merge
|
||||
- ✅ Require 1 approval
|
||||
- ✅ Dismiss stale approvals
|
||||
- ✅ Require CI to pass (where ci exists)
|
||||
- ✅ Block force pushes
|
||||
- ✅ block branch deletion
|
||||
|
||||
| Rule | Status | Repositories |
|
||||
|------|--------|--------------|
|
||||
### Default Reviewers
|
||||
- @perplexity - All repositories (QA gate)
|
||||
- @Timmy - hermes-agent (owner gate)
|
||||
|
||||
### Implementation Status
|
||||
- [x] hermes-agent
|
||||
- [x] the-nexus
|
||||
- [x] timmy-home
|
||||
- [x] timmy-config
|
||||
|
||||
### CI Status
|
||||
- hermes-agent: ✅ ci enabled
|
||||
- the-nexus: ⚠ ci pending (#915)
|
||||
- timmy-home: ❌ No ci
|
||||
- timmy-config: ❌ No ci
|
||||
| Require PR for merge | ✅ Enabled | hermes-agent, the-nexus, timmy-home, timmy-config |
|
||||
| Required approvals | ✅ 1+ required | All |
|
||||
| Dismiss stale approvals | ✅ Enabled | All |
|
||||
|
||||
Reference in New Issue
Block a user