Skip to content

docs: enforce branch protection rules#30

Merged
abrichr merged 2 commits intomainfrom
fix/enforce-branch-protection
Feb 24, 2026
Merged

docs: enforce branch protection rules#30
abrichr merged 2 commits intomainfrom
fix/enforce-branch-protection

Conversation

@abrichr
Copy link
Member

@abrichr abrichr commented Feb 24, 2026

Summary

  • Added mandatory branch/PR rule to CLAUDE.md
  • Enabled enforce_admins on GitHub branch protection for this repo
  • This prevents any user (including admins) from pushing directly to main

Context

Several commits were pushed directly to main in a previous session:

  • 120c903 — Modal training fixes (serialized, auto-create volume, --force, scoping, deps, --jsonl)
  • 57e5c5f — Modal inference serving (call_inference API, serve CLI)
  • 88e4c09 — Inference container image fixes (transformers pinning, torchvision, multi-modal messages)

This change ensures that can never happen again.

Test plan

  • Verify enforce_admins: true via GitHub API
  • Verify direct push to main is rejected

🤖 Generated with Claude Code

@abrichr abrichr force-pushed the fix/enforce-branch-protection branch from b8fd491 to 891d5c2 Compare February 24, 2026 20:59
abrichr and others added 2 commits February 24, 2026 16:32
Adds explicit instruction that all changes must go through feature
branches and pull requests. enforce_admins has been enabled on GitHub
to prevent admin bypass of branch protection.

Co-Authored-By: Claude Opus 4.6 <[email protected]>
Fixes ruff F401 lint error on modal_cloud.py.

Co-Authored-By: Claude Opus 4.6 <[email protected]>
@abrichr abrichr force-pushed the fix/enforce-branch-protection branch from 891d5c2 to 6e584f1 Compare February 24, 2026 21:32
@abrichr abrichr merged commit afad981 into main Feb 24, 2026
4 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant