Skip to content

Commit 0c21fdd

Browse files
committed
docs: point the PR checklist at the gate the workflow documents
The checklist asked for `composer ci`, which is not the pre-push gate. `ci` runs cgl, phpstan, unit, integration and fuzzy; `make gate` runs cgl, phpstan, unit, fuzzy, rector pinned to PHP 8.2 and functional against sqlite, plus the CHANGELOG check. Rector and the functional suite are exactly the two the Makefile comment names as the ones that get skipped by hand and are then found by the CI matrix a push later. So the box a contributor ticked certified a narrower set than the workflow in AGENTS.md prescribes, and the difference was invisible at the point of ticking it. Both targets stay — they answer different questions — but the checklist now names the one that gates a push. Assisted-by: claude-code:claude-opus-5 Agent-Session: https://claude.ai/code/session_015QXXkquh2eQNBiTYA39Wss host=32116e Signed-off-by: Sebastian Mendel <github@sebastianmendel.de>
1 parent 6fe501d commit 0c21fdd

1 file changed

Lines changed: 1 addition & 1 deletion

File tree

.github/PULL_REQUEST_TEMPLATE.md

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -17,7 +17,7 @@
1717
## Checklist
1818

1919
- [ ] My code follows the project's coding standards
20-
- [ ] I have run `composer ci` and all checks pass
20+
- [ ] I have run `make gate` and all checks pass (not `composer ci` — that set omits Rector and the functional suite)
2121
- [ ] I have added tests that prove my fix/feature works
2222
- [ ] I have updated the documentation accordingly
2323
- [ ] I have added a `CHANGELOG.md` entry under `## [Unreleased]`

0 commit comments

Comments
 (0)