-
Notifications
You must be signed in to change notification settings - Fork 2.7k
Commit
This commit does not belong to any branch on this repository, and may belong to a fork outside of the repository.
Merge pull request #154 from ai16z/sirkitree-patch-1
Create pull_request_template.md
- Loading branch information
Showing
1 changed file
with
70 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,70 @@ | ||
| <!-- Use this template by filling in information and copy and pasting relevant items out of the html comments. --> | ||
|
|
||
| # Relates to: | ||
| <!-- LINK TO ISSUE OR TICKET --> | ||
|
|
||
| <!-- This risks section is to be filled out before final review and merge. --> | ||
| # Risks | ||
| <!-- | ||
| Low, medium, large. List what kind of risks, and what could be effected. | ||
| --> | ||
|
|
||
| # Background | ||
| ## What does this PR do? | ||
|
|
||
| ## What kind of change is this? | ||
| <!-- | ||
| Bug fixes (non-breaking change which fixes an issue) | ||
| Improvements (misc. changes to existing features) | ||
| Features (non-breaking change which adds functionality) | ||
| Updates (new versions of included code) | ||
| --> | ||
|
|
||
| <!-- This "Why" section is most relevant if there is no linked issue explaining why. If there is a related issue it might make sense to skip this why section. --> | ||
| <!-- | ||
| ## Why are we doing this? Any context or related work? | ||
| --> | ||
|
|
||
| # Documentation changes needed? | ||
| <!-- | ||
| My changes do not require a change to the project documentation. | ||
| My changes require a change to the project documentation. | ||
| If a docs change is needed: I have updated the documentation accordingly. | ||
| --> | ||
|
|
||
| <!-- Please show how you tested the PR. This will really help if the PR needs to be retested, and probably help the PR get merged quicker. --> | ||
| # Testing | ||
| ## Where should a reviewer start? | ||
|
|
||
| ## Detailed testing steps | ||
| <!-- | ||
| None, automtated tests are fine. | ||
| --> | ||
|
|
||
| <!-- | ||
| - As [anon/admin], go to [link] | ||
| - [do action] | ||
| - verify [result] | ||
| --> | ||
|
|
||
| <!-- If there is a UI change, please include before and after screenshots or videos. This will speed up PRs being merged. It is extra nice to annotate screenshots with arrows or boxes pointing out the differences. --> | ||
| <!-- | ||
| ## Screenshots | ||
| ### Before | ||
| ### After | ||
| --> | ||
|
|
||
| <!-- If there is anything about the deploy, please make a note. --> | ||
| <!-- | ||
| # Deploy Notes | ||
| --> | ||
|
|
||
| <!-- Copy and paste commandline output. --> | ||
| <!-- | ||
| ## Database changes | ||
| --> | ||
|
|
||
| <!-- If there is something more than the automated steps, please specifiy deploy instructions. --> | ||
| <!-- | ||
| ## Deployment instructions | ||
| --> |