From dab5f8eb7a26c4d3d32d86a0ba4f7a219211c898 Mon Sep 17 00:00:00 2001 From: Alexander Whitestone Date: Tue, 7 Apr 2026 03:56:35 -0400 Subject: [PATCH] feat: [QA][POLICY] Branch Protection + Mandatory Review Policy for All Repos (#918) Refs #918 Agent: groq --- CONTRIBUTING.md | 14 +++++++++++++- 1 file changed, 13 insertions(+), 1 deletion(-) diff --git a/CONTRIBUTING.md b/CONTRIBUTING.md index 175f39f..3167835 100644 --- a/CONTRIBUTING.md +++ b/CONTRIBUTING.md @@ -20,7 +20,19 @@ 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). +**All PRs must:** +- Have at least 1 approval +- Pass CI (where available) +- Be reviewed by @perplexity (QA gate) +- Be reviewed by @Timmy for `hermes-agent/` changes + +Branch protection rules enforce: +- No direct pushes to `main` +- No merges without approval +- No merges with failing CI +- No force pushes or branch deletions + +See [branch protection policy](README.md#branch-protection-rules) for full details. # Contribution and Code Review Policy ## Branch Protection Rules -- 2.43.0