-
Notifications
You must be signed in to change notification settings - Fork 9
Commit
This commit does not belong to any branch on this repository, and may belong to a fork outside of the repository.
- Loading branch information
1 parent
146725f
commit e03485c
Showing
1 changed file
with
23 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,23 @@ | ||
## Summary | ||
|
||
> Developers should provide a summary of the proposed changes here and include "closes #<ISSUE NUMBER>" if this addresses an open issue. | ||
> Developers should include validation test reports demonstrating that the reported errors have been solved. | ||
## Reviewer Checklist | ||
|
||
> Reviewers should tick the following boxes before approving and merging the PR. | ||
- [ ] The PR targets the `develop` branch | ||
- [ ] The PR is assigned to the developer | ||
- [ ] The PR has appropriate labels | ||
- [ ] The PR is included in appropriate projects and/or milestones | ||
- [ ] The PR includes a clear description of the proposed changes | ||
- [ ] If the PR addresses an open issue the description includes "closes #<ISSUE NUMBER>" | ||
- [ ] The code and documentation style match the current standards | ||
- [ ] Documentation has been added/updated consistently with the code | ||
- [ ] All CI tests are passing | ||
- [ ] API docs have been built and checked at least once (if relevant) | ||
- [ ] All changed files have been checked and comments provided to the developer | ||
- [ ] All of the reviewer's comments have been satisfactorily addressed by the developer | ||
- [ ] All Validation tests for the reported error are passing | ||
|