Revise PR template for clarity and structure
Updated the pull request template to clarify guidelines and checklist items.
This commit is contained in:
@@ -1,19 +1,17 @@
|
|||||||
<!--
|
<!--
|
||||||
Please follow this template. The PR template applies whether you opened the PR via the web UI, `gh pr create`, or any other tool.
|
Please follow this template. The PR template applies whether you opened the PR via the web UI, `gh pr create`, or any other tool.
|
||||||
|
|
||||||
If you're fixing an objective bug or a small focused issue, this should be quick. Big PRs without a prior issue or design discussion are likely to be closed or scoped down. See CONTRIBUTING.md.
|
See CONTRIBUTING.md.
|
||||||
-->
|
-->
|
||||||
|
|
||||||
### Linked issue
|
### Linked issue
|
||||||
|
|
||||||
Closes #
|
Closes #
|
||||||
|
|
||||||
<!-- Bug fixes and behavior changes should reference an issue. Pure docs and refactors can skip this. -->
|
|
||||||
|
|
||||||
### Type of change
|
### Type of change
|
||||||
|
|
||||||
- [ ] Bug fix
|
- [ ] Bug fix
|
||||||
- [ ] New feature (with prior issue + design alignment)
|
- [ ] New feature
|
||||||
- [ ] Refactor / code improvement
|
- [ ] Refactor / code improvement
|
||||||
- [ ] Docs
|
- [ ] Docs
|
||||||
|
|
||||||
@@ -29,15 +27,13 @@ This is the section I read most carefully. I need to see that *you* tested this,
|
|||||||
- For UI changes: a screenshot or short video on every affected platform (mobile, web, desktop). UI claims without visual proof are not enough.
|
- For UI changes: a screenshot or short video on every affected platform (mobile, web, desktop). UI claims without visual proof are not enough.
|
||||||
- For behavior changes: the actual steps you ran, and what you observed.
|
- For behavior changes: the actual steps you ran, and what you observed.
|
||||||
- For bug fixes: how you reproduced the bug before, and confirmed it's fixed after.
|
- For bug fixes: how you reproduced the bug before, and confirmed it's fixed after.
|
||||||
|
|
||||||
AI-generated PR descriptions are fine in principle. AI-generated *verification claims* with no actual testing behind them are not, and they're easy to spot.
|
|
||||||
-->
|
-->
|
||||||
|
|
||||||
### Checklist
|
### Checklist
|
||||||
|
|
||||||
- [ ] One focused change. Unrelated cleanups split out.
|
- [ ] One focused change
|
||||||
- [ ] `npm run typecheck` passes
|
- [ ] `npm run typecheck` passes
|
||||||
- [ ] `npm run lint` passes
|
- [ ] `npm run lint` passes
|
||||||
- [ ] `npm run format` ran (Biome)
|
- [ ] `npm run format` passes
|
||||||
- [ ] UI changes include screenshots or video for every affected platform
|
- [ ] QA evidence
|
||||||
- [ ] Tests added or updated where it made sense
|
- [ ] Tests added or updated where it made sense
|
||||||
|
|||||||
Reference in New Issue
Block a user