Skip to content

docs: give a pull request the shape of a review #2

New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Open
BryanZaneee wants to merge 1 commit into
base: develop
Choose a base branch
from
Open

docs: give a pull request the shape of a review #2

Changes from all commits
Commits
Show all changes
1 commit
Select commit
File filter

Filter by extension

Filter by extension
Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
30 changes: 30 additions & 0 deletions .github/pull_request_template.md
View file
Open in desktop
Original file line number Diff line number Diff line change
@@ -0,0 +1,30 @@
## Summary

<!-- One or two sentences: what this PR does. -->

## Why

<!-- The problem or need. Link the issue: Closes #N -->

## Changes

<!-- The notable ones, not a file list. -->

## Testing

<!-- Commands run and their result. `npm test` at minimum. Paste the numbers
that matter (counts, ratios, timings) rather than "tests pass". -->

## Checklist

- [ ] Docs moved in the same commit as the code (CLAUDE.md's table)
- [ ] Goldens unchanged, or re-baselined deliberately with the diff read
- [ ] `test/generic.test.mjs` green: nothing target-specific in `src/` or `bin/`
- [ ] No new dependency, or one discussed and justified above
- [ ] ROADMAP.md boxes ticked; phase status and header count updated
- [ ] Nothing modelled is drawn as observed; no number the tool does not measure

## ADR

- [ ] An architectural decision changed → `docs/adr/NNNN-*.md` added or updated, linked here
- [ ] Not applicable
Loading