commit-and-pr
Commit and Pull Request
Creates commits and PRs that follow Arena's contributor guidelines.
Commit
-
Run the
pre-commit-checkskill (orpre-commit run --all-fileson the host — pre-commit is not installed in the container) until the working tree is clean. Do not commit through hook failures. -
Stage specific files rather than using
git add -A— this avoids accidentally committing dotfiles, credentials, or large untracked artifacts. -
Sign off the commit (per
CONTRIBUTING.md— Arena currently requires DCO sign-off; the policy is on the books but not yet CI-enforced):git commit -s -m "<subject>" -
Subject line: imperative mood, ~50 characters, no trailing period.
- Good:
Fix attribute access on wrapped env - Bad:
Fixed the attribute access on the wrapped env.(past tense, trailing period)
- Good:
-
Body (when needed): blank line after the subject, wrap at 72 chars, explain what and why — not how (the diff shows that).
-
No AI attribution lines. Do not include
Co-Authored-By: Claude...,Generated with Claude Code, or similar footers. The DCO sign-off is the only required trailer.