Skip to content

docs: enforce branch protection rules in CLAUDE.md#199

Merged
github-actions[bot] merged 1 commit into
developfrom
fix/enforce-branch-protection
Apr 23, 2026
Merged

docs: enforce branch protection rules in CLAUDE.md#199
github-actions[bot] merged 1 commit into
developfrom
fix/enforce-branch-protection

Conversation

@tomymaritano

@tomymaritano tomymaritano commented Apr 23, 2026 •
edited by coderabbitai Bot
Loading

Copy link
Copy Markdown
Collaborator

Summary

  • Add explicit rule: never push directly to develop or main
  • Add instruction for Claude Code to always create feature branches
  • Add "always branch from develop" and "clean up after merge" to hygiene rules

🤖 Generated with Claude Code

Summary by CodeRabbit

  • No user-facing changes
    • Internal documentation update to development workflow and branch governance guidelines. This does not affect end-user functionality or features.

@tomymaritano @claude
docs: enforce branch protection rules in CLAUDE.md
3b0cd18 
- Add explicit "NEVER push directly to develop or main" rule
- Add "Claude Code must also follow this" instruction
- Add "Always branch from develop" to hygiene rules
- Add "Clean up after merge" rule

Co-Authored-By: Claude Opus 4.6 (1M context) <noreply@anthropic.com>
@vercel

vercel Bot commented Apr 23, 2026 •
edited
Loading

Copy link
Copy Markdown

The latest updates on your projects. Learn more about Vercel for GitHub.

Project Deployment Actions Updated (UTC)
readide Error Error Apr 23, 2026 5:10pm

Request Review

@github-actions github-actions Bot enabled auto-merge (squash) April 23, 2026 17:10
@coderabbitai

coderabbitai Bot commented Apr 23, 2026 •
edited
Loading

Copy link
Copy Markdown

Caution

Review failed

Pull request was closed or merged during review

📝 Walkthrough

Walkthrough

Documentation update to CLAUDE.md enforcing stricter Git Flow branch governance by prohibiting direct pushes to develop, adding mandatory branch protection rules, specifying PR-only workflows, and detailing branch hygiene practices including post-merge cleanup.

Changes

Cohort / File(s) Summary
Git Flow Documentation
CLAUDE.md		 Added branch protection rules enforcement in diagram and new "Branch Protection Rules (Critical)" section specifying PR-only work and mandatory PR flow. Enhanced "Branch Hygiene" with concrete branching commands, "never push directly to develop" reinforcement, and post-merge local/remote branch cleanup steps.

Estimated code review effort

🎯 1 (Trivial) | ⏱️ ~3 minutes

🚥 Pre-merge checks | ✅ 5
✅ Passed checks (5 passed)
Check name Status Explanation
Description Check ✅ Passed Check skipped - CodeRabbit’s high-level summary is enabled.
Title check ✅ Passed The title accurately describes the main change: adding and enforcing branch protection rules in the CLAUDE.md documentation file.
Docstring Coverage ✅ Passed No functions found in the changed files to evaluate docstring coverage. Skipping docstring coverage check.
Linked Issues check ✅ Passed Check skipped because no linked issues were found for this pull request.
Out of Scope Changes check ✅ Passed Check skipped because no linked issues were found for this pull request.

✏️ Tip: You can configure your own custom pre-merge checks in the settings.

✨ Finishing Touches
🧪 Generate unit tests (beta)
  • Create PR with unit tests
  • Commit unit tests in branch fix/enforce-branch-protection

Thanks for using CodeRabbit! It's free for OSS, and your support helps us grow. If you like it, consider giving us a shout-out.

❤️ Share

Comment @coderabbitai help to get the list of available commands and usage tips.

@github-actions github-actions Bot merged commit 19bf2ca into develop Apr 23, 2026
13 of 15 checks passed
@tomymaritano tomymaritano mentioned this pull request Apr 23, 2026
Merged
1 task
tomymaritano added a commit that referenced this pull request Apr 23, 2026
@tomymaritano @claude @dependabot
fix: sync main into develop for clean release (#205)
b5d2733 
## Summary
Merges main into develop to resolve all conflicts so PR #202 (release)
can merge cleanly.

All 5 conflicts resolved keeping develop's version (which has all review
fixes from PRs #199-#204).

## Test plan
- [x] `pnpm typecheck` — 17/17 pass

🤖 Generated with [Claude Code](https://claude.com/claude-code)

Signed-off-by: dependabot[bot] <support@github.com>
Co-authored-by: Claude Opus 4.6 <noreply@anthropic.com>
Co-authored-by: dependabot[bot] <49699333+dependabot[bot]@users.noreply.github.com>
@tomymaritano tomymaritano deleted the fix/enforce-branch-protection branch April 23, 2026 20:35
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

No reviews

Assignees

No one assigned

Labels

size/S

Projects

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

1 participant

@tomymaritano