-
Notifications
You must be signed in to change notification settings - Fork 5
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
Showing
1 changed file
with
14 additions
and
4 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 |
---|---|---|
@@ -1,8 +1,18 @@ | ||
## Changes | ||
- List all major changes made here | ||
# Change | ||
|
||
_Please remove everything in italics._ | ||
|
||
_Add a brief and concise imperative sentence on what this PR does._ | ||
_Example: Edit PR template to improve efficiency_ | ||
|
||
## Change reason | ||
List the reason you are making the change here. | ||
|
||
_Provide a brief description based on the type of change._ | ||
|
||
1. _Functionality - explain the problem, why the solution was chosen, and any specifics that might be worth highlighting_ | ||
|
||
2. _Refactoring - why there was a need for the change, how the changes address this problem, and steps to take to avoid the same mistakes (better documentation, another step in PR reviews, simplification of system design)_ | ||
|
||
## Comments | ||
- Add any other comments about the PR here | ||
|
||
_Add any other comments about the PR here. In particular add information relating where a reviewer should start and any specific concerns that you came across._ |