-
Notifications
You must be signed in to change notification settings - Fork 117
Commit
This commit does not belong to any branch on this repository, and may belong to a fork outside of the repository.
chore(templates): update pull request template
- Loading branch information
Showing
1 changed file
with
19 additions
and
0 deletions.
There are no files selected for viewing
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Original file line number | Diff line number | Diff line change |
---|---|---|
@@ -0,0 +1,19 @@ | ||
|
||
--- | ||
|
||
### Acceptance Checklist | ||
|
||
- [ ] **Story**: Code is focused on the linked stories and solves a problem | ||
- One of: | ||
- [ ] **For Bugs**: A unit test is added or an existing one modified | ||
- [ ] **For Features**: New unit tests are added covering the new functions or modifications | ||
- [ ] Code Documentation changes are included for public interfaces and important / complex additions | ||
- [ ] External Documentation is included for API changes, or other external facing interfaces | ||
|
||
### Review Checklist | ||
|
||
- [ ] The code does not duplicate existing functionality that exists elsewhere | ||
- [ ] The code has been linted and follows team practices and style guidelines | ||
- [ ] The changes in the PR are relevant to the title | ||
- changes not related should be moved to a different PR | ||
- [ ] All errors or error handling is actionable, and informs the viewer on how to correct it |