Skip to content
Merged
Changes from all commits
Commits
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
34 changes: 27 additions & 7 deletions .github/PULL_REQUEST_TEMPLATE.md
Original file line number Diff line number Diff line change
Expand Up @@ -7,22 +7,42 @@
### How
<!-- Approach taken. Key design decisions and trade-offs. -->

### Testing
<!-- What was tested and how. "npm run preflight" output or CI green. -->
---

### PR Readiness Checklist

> The PR readiness bot will validate these automatically after push.
> Check each item before requesting review. See [CONTRIBUTING.md](../CONTRIBUTING.md) for full details.

#### Branch & Commit
- [ ] Branch created from `dev` (not `main`)
- [ ] Branch is up to date with `dev` (`git fetch upstream && git rebase upstream/dev`)
- [ ] PR is **not** in draft mode (mark ready when checks pass)
- [ ] Commit history is clean (squash fixups before review)

#### Build & Test
- [ ] `npm run build` passes
- [ ] `npm test` passes (all tests green)
- [ ] For migration PRs (>20 files): test output summary included below
- [ ] `npm run lint` passes (type check clean)
Comment thread
diberry marked this conversation as resolved.
Resolved
- [ ] `npm run lint:eslint` passes
- [ ] For migration PRs (>20 files): include test output summary in PR description

Comment thread
diberry marked this conversation as resolved.
Resolved
#### Changeset
- [ ] Changeset added via `npx changeset add` (if `packages/squad-sdk/src/` or `packages/squad-cli/src/` changed)
- [ ] Or direct `CHANGELOG.md` entry (maintainers only — write-protected for external contributors)
- [ ] Or `skip-changelog` label applied (if no user-facing changes)

### Docs
<!-- What documentation was updated. "N/A" only if truly no user-facing change. -->
- [ ] CHANGELOG.md entry (if packages/squad-sdk/src/ or packages/squad-cli/src/ changed)
#### Docs
<!-- "N/A" only if truly no user-facing change. -->
- [ ] README section updated (if new feature/module)
- [ ] Docs feature page (if new user-facing capability)

### Exports
#### Exports
<!-- For SDK changes only. "N/A" if no new modules. -->
- [ ] package.json subpath exports updated (if new module)

---

### Breaking Changes
<!-- Any backward-incompatible changes. "None" if clean. -->

Expand Down
Loading