Skip to content

chore: add PR template and required CI checks (V2-719, V2-720)#12

Open
jacderida wants to merge 1 commit into
mainfrom
chrisoneil/v2-719-add-the-standard-pr-template-across-all-six-crate-repos
Open

chore: add PR template and required CI checks (V2-719, V2-720)#12
jacderida wants to merge 1 commit into
mainfrom
chrisoneil/v2-719-add-the-standard-pr-template-across-all-six-crate-repos

Conversation

@jacderida

Copy link
Copy Markdown
Contributor

Linear issue

Risk tier

  • T0 — docs / tooling / CI / pure UX-output. Repo CI only.
  • T1 — client-only, no network-facing behavior change. CI + prod compat smoke.
  • T2 — node/client logic with behavioral surface, no protocol/format/economics change. Dev testnet + ADR.
  • T3 — protocol / storage format / payments / routing. T2 evidence + adversarial testing.

Compatibility

  • Wire: none
  • Storage: none
  • API: none

Semver impact

  • breaking
  • feature
  • fix

Test evidence

check_pr.py was exercised locally across its cases: missing Linear reference
fails linear; a key in the title/body/branch passes; a body without the
template fails template; an rc-* base is a no-op pass; a fully-filled template
passes; and the failure modes (no/duplicate tier or semver box, empty section,
Tier 2/3 without an ADR link) each fail with a specific message. This PR's own
description is validated by the new checks.

New dependency

none

ADR

n/a

Mitigation / rollback

Revert this PR; the template and checks are additive and carry no runtime effect.

Add the standard pull request template (V2-719) and the CI enforcement that
makes a linked Linear issue and the template fields required on every PR
(V2-720). Part of the release-process extensions; identical across the six
crate repos.

- .github/PULL_REQUEST_TEMPLATE.md: eight-field template with Risk tier and
  Semver impact checkboxes, stable headings as grep anchors for the
  train-manifest verification.
- agent instruction files (CLAUDE.md / AGENTS.md): instruct agents to fill the
  template on PR open.
- .github/workflows/pr-checks.yml + .github/scripts/check_pr.py: the
  linear-link (main + rc-*) and pr-template (main) required status checks.

V2-719
V2-720

Co-Authored-By: Claude Opus 4.8 (1M context) <noreply@anthropic.com>
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