Skip to content

Commit

Permalink
chore: adds reviewer checklist to PR template
Browse files Browse the repository at this point in the history
Signed-off-by: Jennifer Power <[email protected]>
  • Loading branch information
jpower432 committed Nov 1, 2023
1 parent e93a4e1 commit 00f81fc
Showing 1 changed file with 11 additions and 2 deletions.
13 changes: 11 additions & 2 deletions .github/pull_request_template.md
Original file line number Diff line number Diff line change
Expand Up @@ -11,7 +11,7 @@ Fixes # (issue)
- [ ] Bug fix (non-breaking change which fixes an issue)
- [ ] New feature (non-breaking change which adds functionality)
- [ ] Breaking change (fix or feature that would cause existing functionality to not work as expected)
- [ ] This change requires a documentation update
- [ ] Documentation only change

## How has this been tested?

Expand All @@ -27,7 +27,7 @@ Fixes # (issue)
- Toolchain:
- SDK:

## Checklist
## Author Checklist

- [ ] My code follows the style guidelines of this project
- [ ] I have performed a self-review of my own code
Expand All @@ -37,3 +37,12 @@ Fixes # (issue)
- [ ] I have added tests that prove my fix is effective or that my feature works
- [ ] New and existing unit tests pass locally with my changes
- [ ] Any dependent changes have been merged and published in downstream modules

## Reviewer Checklist

- [ ] Implementation matches the proposed design, or proposal is updated to match implementation
- [ ] Sufficient unit test coverage
- [ ] Sufficient end-to-end test coverage
- [ ] Docs updated or added, if needed
- [ ] Commit messages sensible and descriptive
- [ ] Any tests specified as manual have been performed and verified as successful

0 comments on commit 00f81fc

Please sign in to comment.