Skip to content

Keep PR branch commits to one line - #2

Merged
saaguero merged 1 commit into
mainfrom
one-line-commits
Aug 31, 2026
Merged

saaguero merged 1 commit into
mainfrom
one-line-commits

Conversation

@saaguero

Copy link
Copy Markdown
Owner

A squash merge discards branch commit bodies, so anything written there is lost to main's history. The commits section of AGENTS.md now puts the whole story in the PR title and body (which the squash inherits) and keeps branch commits to their subject line, matching how the merged history actually reads.

🤖 Generated with Claude Code

Co-Authored-By: Claude Fable 5 <noreply@anthropic.com>
@saaguero
saaguero merged commit 0bbf257 into main Aug 31, 2026
1 of 2 checks passed
@saaguero
saaguero deleted the one-line-commits branch August 31, 2026 19:34
@coderabbitai

coderabbitai Bot commented Aug 31, 2026 •

Copy link
Copy Markdown

Review Change Stack

Caution

Review failed

The pull request is closed.

ℹ️ Recent review info
⚙️ Run configuration

Configuration used: Path: .coderabbit.yaml

Review profile: CHILL

Plan: Pro Plus

Run ID: 84190f6b-3e31-4127-bc2f-381435f4b592

📥 Commits

Reviewing files that changed from the base of the PR and between defcf07 and b618a93.

📒 Files selected for processing (1)
  • AGENTS.md

📝 Walkthrough

Walkthrough

The contribution guidance now separates branch commit subjects from PR metadata. It defines squash-merge commit content, requires co-author trailers on agent-authored commits, and scopes the single-concern rule to PRs.

Changes

Contribution Guidance

Layer / File(s) Summary
Commit and pull request rules
AGENTS.md
The guidance distinguishes branch commit subjects from PR titles and bodies. It requires co-author trailers on agent-authored branch commits, uses PR metadata for squash-merged commits, and applies the single-concern rule per PR.

Estimated code review effort: 1 (Trivial) | ~2 minutes

✨ Finishing Touches
🧪 Generate unit tests (beta)
  • Create PR with unit tests
  • Commit unit tests in branch one-line-commits

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

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