0ccae4c7c5
Some checks failed
CI / validate (pull_request) Failing after 12s
Add docs/pr-reviewer-policy.md defining org-wide rules: - Every PR must have a reviewer assigned before merge - Default reviewer: @perplexity - Rebase or close PRs that drift more than 2 weeks - Close zombie PRs (0 commits ahead of base) Add reference to reviewer policy in CONTRIBUTING.md. Established after PR hygiene audit found 5 open PRs with 0 reviewers across the org, leading to two critical PRs becoming unmergeable after 400+ commits of drift. Refs #916
24 lines
903 B
Markdown
24 lines
903 B
Markdown
# Contributing to the Nexus
|
|
|
|
**Every PR: net ≤ 10 added lines.** Not a guideline — a hard limit.
|
|
Add 40, remove 30. Can't remove? You're homebrewing. Import instead.
|
|
|
|
## Why
|
|
|
|
Import over invent. Plug in the research. No builder trap.
|
|
Removal is a first-class contribution. Baseline: 4,462 lines (2026-03-25). Goes down.
|
|
|
|
## PR Checklist
|
|
|
|
1. **Net diff ≤ 10** (`+12 -8 = net +4 ✅` / `+200 -0 = net +200 ❌`)
|
|
2. **Manual test plan** — specific steps, not "it works"
|
|
3. **Automated test output** — paste it, or write a test (counts toward your 10)
|
|
|
|
Applies to every contributor: human, Timmy, Claude, Perplexity, Gemini, Kimi, Grok.
|
|
Exception: initial dependency config files (requirements.txt, package.json).
|
|
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).
|