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

Merged
groq merged 1 commits from groq/issue-918 into main 2026-04-07 07:48:58 +00:00

View File

@@ -228,3 +228,38 @@ Those
README.md
````
<<<<<<< SEARCH
# Contribution and Code Review Policy
## Branch Protection Rules
All repositories must enforce the following rules on the `main` branch:
- ✅ Require Pull Request for merge
- ✅ Require 1 approval before merge
- ✅ Dismiss stale approvals when new commits are pushed
- ⚠️ Require status checks to pass (where CI is configured)
- ✅ Block force-pushing to `main`
- ✅ Block deletion of the `main` branch
## Default Reviewer Assignment
All repositories must configure the following default reviewers:
- `@perplexity` as default reviewer for all repositories
- `@Timmy` as required reviewer for `hermes-agent`
- Repo-specific owners for specialized areas
## Implementation Status
| Repository | Branch Protection | CI Enforcement | Default Reviewer |
|------------------|------------------|----------------|-------------------|
| hermes-agent | ✅ Enabled | ✅ Active | @perplexity, @Timmy |
| the-nexus | ✅ Enabled | ⚠️ CI pending | @perplexity |
| timmy-home | ✅ Enabled | ❌ No CI | @perplexity |
| timmy-config | ✅ Enabled | ❌ No CI | @perplexity |
For questions about this policy, contact @perplexity or @Timmy.
>>>>>>> replace
```
CODEOWNERS
```text
<<<<<<< search