Skip to content

docs: require agents to respect DNM markers on PRs#3871

Merged
jlowin merged 2 commits intomainfrom
docs/dnm-pr-rule
Apr 12, 2026
Merged

docs: require agents to respect DNM markers on PRs#3871
jlowin merged 2 commits intomainfrom
docs/dnm-pr-rule

Conversation

@jlowin
Copy link
Copy Markdown
Member

@jlowin jlowin commented Apr 12, 2026

Agents reviewing or merging PRs need a clear, unambiguous rule about do-not-merge markers. Authors use these as hard stops — often because a PR is part of a coordinated sequence, has a pending dependency, or is intentionally up for review without being ready to merge. Missing one of these markers can easily result in a premature merge that the author then has to reshape around.

This adds an explicit rule to CLAUDE.md that agents must check title, body, AND labels for all common variations of "do not merge" before merging, and that this check must happen immediately before the merge — not just at triage time, since labels can change mid-session.

jlowin added 2 commits April 12, 2026 12:56
Require agents to respect do-not-merge markers across title, body, and
labels before merging any PR, with explicit variations listed.
@marvin-context-protocol marvin-context-protocol Bot added the documentation Updates to docs, examples, or guides. Primary change is documentation-related. label Apr 12, 2026
@jlowin jlowin merged commit f71680c into main Apr 12, 2026
8 checks passed
@jlowin jlowin deleted the docs/dnm-pr-rule branch April 12, 2026 16:58
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

documentation Updates to docs, examples, or guides. Primary change is documentation-related.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant