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

Merged
groq merged 1 commits from groq/issue-918 into main 2026-04-07 06:31:43 +00:00
2 changed files with 109 additions and 0 deletions

View File

@@ -6,3 +6,58 @@ branch_protection:
require_ci_to_pass: true
block_force_pushes: true
block_deletions: true
repos:
- name: hermes-agent
branch_protection:
required_pull_request_reviews: true
required_status_checks:
- "ci/circleci"
- "security-scan"
required_linear_history: true
required_merge_method: merge
required_pull_request_reviews:
required_approving_review_count: 1
block_force_pushes: true
block_deletions: true
required_owners:
- perplexity
- Timmy
- name: the-nexus
branch_protection:
required_pull_request_reviews: true
required_status_checks: []
required_linear_history: true
required_merge_method: merge
required_pull_request_reviews:
required_approving_review_count: 1
block_force_pushes: true
block_deletions: true
required_owners:
- perplexity
- name: timmy-home
branch_protection:
required_pull_request_reviews: true
required_status_checks: []
required_linear_history: true
required_merge_method: merge
required_pull_request_reviews:
required_approving_review_count: 1
block_force_pushes: true
block_deletions: true
required_owners:
- perplexity
- name: timmy-config
branch_protection:
required_pull_request_reviews: true
required_status_checks: []
required_linear_history: true
required_merge_method: merge
required_pull_request_reviews:
required_approving_review_count: 1
block_force_pushes: true
block_deletions: true
required_owners:
- perplexity

View File

@@ -21,3 +21,57 @@ No other exceptions. Too big? Break it up.
## Reviewer Policy
**Every PR must have a reviewer assigned before merge.** See [docs/pr-reviewer-policy.md](docs/pr-reviewer-policy.md).
# Contribution Guidelines
## Branch Protection Policy
All repositories follow these mandatory rules for the `main` branch:
### Required for All Merges
- [ ] Pull Request must exist for all changes
- [ ] At least 1 approval from reviewer
- [ ] CI checks must pass (where available)
- [ ] No force pushes allowed
- [ ] No direct pushes to main
- [ ] No branch deletion
### Review Requirements by Repository
```yaml
hermes-agent:
required_owners:
- perplexity
- Timmy
the-nexus:
required_owners:
- perplexity
timmy-home:
required_owners:
- perplexity
timmy-config:
required_owners:
- perplexity
```
### CI Status
```text
- 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
3. Get 1+ approvals
4. Ensure CI passes
5. Merge via UI
## Enforcement
These rules are enforced by Gitea branch protection settings. Direct pushes to main will be blocked.
## Abandoned PRs
PRs not updated in >7 days will be labeled "stale" and may be closed after 30 days of inactivity.