The PR template filled in job in pr-checks.yml hard-requires a non-empty ## Test Plan section, and the template's Test Plan scaffolding (cargo-test checkboxes) assumes the PR changes code.
Release PRs (e.g. #854: version bump + changelog roll, no code), docs-only, and CI-only PRs have no test plan in that sense — they either fail the check or fill in boilerplate checkboxes that aren't true.
Fix: rename the section to a generic ## Verification (how was this PR validated — tests for code, CI + changelog verification for releases, rendered output for docs), and have the check accept ## Verification or the legacy ## Test Plan so in-flight PRs keep passing. Checklist wording for CHANGELOG also generalized (a release PR rolls [Unreleased] rather than adding to it).
The
PR template filled injob inpr-checks.ymlhard-requires a non-empty## Test Plansection, and the template's Test Plan scaffolding (cargo-test checkboxes) assumes the PR changes code.Release PRs (e.g. #854: version bump + changelog roll, no code), docs-only, and CI-only PRs have no test plan in that sense — they either fail the check or fill in boilerplate checkboxes that aren't true.
Fix: rename the section to a generic
## Verification(how was this PR validated — tests for code, CI + changelog verification for releases, rendered output for docs), and have the check accept## Verificationor the legacy## Test Planso in-flight PRs keep passing. Checklist wording for CHANGELOG also generalized (a release PR rolls[Unreleased]rather than adding to it).