-
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.
feat(github): add pull request template
- Loading branch information
Showing
1 changed file
with
17 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,17 @@ | ||
# Description | ||
|
||
Please provide a paragraph or two giving a summary of the change, including relevant motivation and context. | ||
|
||
# Checklist: | ||
|
||
- [ ] I have reviewed my diff in github, line by line. | ||
- [ ] Every change is related to the PR description. | ||
- [ ] I have [linked](https://docs.github.com/en/issues/tracking-your-work-with-issues/linking-a-pull-request-to-an-issue) this pull request to the issue(s) that it resolves. | ||
- [ ] There are no unexpected formatting changes, superfluous debug logs, or commented-out code. | ||
- [ ] There are no circuit changes, OR specifications in `/markdown/specs` have been updated. | ||
- [ ] There are no circuit changes, OR a cryptographer has been assigned for review. | ||
- [ ] I've updated any terraform that needs updating (e.g. environment variables) for deployment. | ||
- [ ] The branch has been rebased against the head of its merge target. | ||
- [ ] I'm happy for the PR to be merged at the reviewer's next convenience. | ||
- [ ] New functions, classes, etc. have been documented according to the doxygen comment format. Classes and structs must have `@brief` describing the intended functionality. | ||
- [ ] If existing code has been modified, such documentation has been added or updated. |