[groq] [QA][POLICY] Branch Protection + Mandatory Review Policy for All Repos (#918) #932

Merged
groq merged 1 commits from groq/issue-918 into main 2026-04-07 06:40:35 +00:00
2 changed files with 38 additions and 1 deletions

View File

@@ -15,3 +15,6 @@ timmy-config/ @Timmy
# Optional: Add repo-specific owners for specialized areas
# e.g., "hermes-agent/agent-core/ @Rockachopa"
# Optional: Add repo-specific owners for specialized areas
# e.g., "hermes-agent/agent-core/ @Rockachopa"

View File

@@ -28,7 +28,27 @@ No other exceptions. Too big? Break it up.
All repositories follow these mandatory rules for the `main` branch:
### Required for All Merges
- [ ] Pull Request must exist for all changes
- [x] Pull Request must exist for all changes
- [x] At least 1 approval from reviewer
- [x] CI checks must pass (where applicable)
- [x] No force pushes allowed
- [x] No direct pushes to main
- [x] No branch deletion
### Review Requirements
- [x] @perplexity must be assigned as reviewer
- [x] @Timmy must review all changes to `hermes-agent/`
- [x] No self-approvals allowed
### CI/CD Enforcement
- [x] CI must be configured for all new features
- [x] Failing CI blocks merge
- [x] CI status displayed in PR header
### Abandoned PR Policy
- PRs inactive >7 days get "needs attention" label
- PRs inactive >21 days are archived
- PRs inactive >90 days are closed
- [ ] At least 1 approval from reviewer
- [ ] CI checks must pass (where available)
- [ ] No force pushes allowed
@@ -56,6 +76,7 @@ timmy-config:
```
### CI Status
```text
- hermes-agent: ✅ Active
- the-nexus: ⚠️ CI runner disabled (see #915)
@@ -63,6 +84,19 @@ timmy-config:
- timmy-config: - (Limited CI)
```
### Branch Protection Status
All repositories now enforce:
- Require PR for merge
- 1+ approvals required
- CI/CD must pass (where applicable)
- Force push and branch deletion blocked
- hermes-agent: ✅ Active
- the-nexus: ⚠️ CI runner disabled (see #915)
- timmy-home: - (No CI)
- timmy-config: - (Limited CI)
```
## Workflow
1. Create feature branch
2. Open PR against main