Skip to content

chore: replace the org PR template with a merge brief - #12

Merged
JuaniRios merged 1 commit into
mainfrom
chore/merge-brief-pr-template
Sep 29, 2026
Merged

JuaniRios merged 1 commit into
mainfrom
chore/merge-brief-pr-template

Conversation

@JuaniRios

@JuaniRios JuaniRios commented Sep 29, 2026 •

Copy link
Copy Markdown
Contributor

Replaces the org default PR template with a merge brief: live effect, risk, decisions, proof, and rollout, written for the person who approves the merge. Agents now review the code line by line, so the body stops explaining the code.

Live effect: new PRs opened on GitHub start from this template in every ST0x repo without its own template · Risk: low (no money path, no keys or IAM, docs only, easy to undo) · Ships: on merge · Blocks: ST0x-Technology/st0x.issuance#428, ST0x-Technology/st0x.liquidity#1576, ST0x-Technology/event-sorcery#60

Decisions

  • One org template, not a copy per repo, so the format has one source. The three PRs above delete the repo copies that would override it. pull_request_template.md
  • Drops the Checks list. CI shows the same things, and a ticked box proves nothing.

Risks

  • gt submit only reads templates inside the repo, so PRs opened from the Graphite CLI start with an empty body. Most ST0x repos already work that way.

Proof

  • Tried the format on a real infra PR. The body went from 456 words to about 160, and it gained the plan result and a rollback trap that the old body missed.
  • Not verified: that GitHub loads the template on a new PR. That only shows after the merge.

🤖 Generated with Claude Code

The template now asks for what the merge approver needs: live effect,
risk, decisions, proof, and rollout. Agents review the code line by line,
so the body no longer explains the code.

Co-Authored-By: Claude Opus 5.5 <noreply@anthropic.com>

Copy link
Copy Markdown
Contributor Author

This stack of pull requests is managed by Graphite. Learn more about stacking.

@JuaniRios
JuaniRios marked this pull request as ready for review September 29, 2026 19:39
@JuaniRios
JuaniRios merged commit 65d97d0 into main Sep 29, 2026
4 of 6 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