docs: add PR size limits to CLAUDE.md
Enforce maximum 5 files and 30 lines changed per PR to keep reviews focused and maintainable. Split by layer (frontend, backend, config) when exceeding limits. Co-Authored-By: Claude Sonnet 4.6 <noreply@anthropic.com>
This commit is contained in:
parent
ee3c42a17f
commit
39ed29c0c4
1 changed files with 13 additions and 0 deletions
13
CLAUDE.md
13
CLAUDE.md
|
|
@ -125,6 +125,19 @@ The authenticated email comes from `securityIdentity.getPrincipal().getName()` a
|
||||||
|
|
||||||
This ensures code review, clear history, and traceability.
|
This ensures code review, clear history, and traceability.
|
||||||
|
|
||||||
|
### PR Size Limits
|
||||||
|
|
||||||
|
To keep PRs focused and reviewable, enforce these limits **per PR**:
|
||||||
|
- **Maximum 5 files changed**
|
||||||
|
- **Maximum 30 lines changed** (added, modified, or deleted combined)
|
||||||
|
|
||||||
|
**If exceeding these limits:** Split into multiple PRs. **Prefer splitting by layer:**
|
||||||
|
- One PR for backend changes (`backend/`)
|
||||||
|
- One PR for frontend changes (`frontend/`)
|
||||||
|
- One PR for config/infra (`docker-compose.yml`, `.env`, etc.)
|
||||||
|
|
||||||
|
This keeps code review focused, reduces merge conflicts, and makes history easier to bisect.
|
||||||
|
|
||||||
## Skills
|
## Skills
|
||||||
|
|
||||||
Skills do projeto em `.claude/skills/`:
|
Skills do projeto em `.claude/skills/`:
|
||||||
|
|
|
||||||
Loading…
Reference in a new issue