Written by Table of contents
Define “done” up front
In the issue description, include:
- Expected behavior and edge cases
- How to test (manual steps and/or automated tests)
- Any UX requirements (screenshots, copy, responsiveness)
Use a repeatable review checklist
- Scope: matches what was agreed
- Quality: readable code, consistent patterns
- Safety: no obvious security or data handling issues
- Tests: added/updated where appropriate
- Docs: updated if behavior changed
Acceptance
Accept/merge when the agreed requirements are met and your validation steps pass.
