GeezerVison is showing github.com — CHANGE: Update pull request template (#1950) · Unity-Technologies/InputSystem@f228c14
File tree
Expand file treeCollapse file tree
Open diff view settings
Expand file treeCollapse file tree
Open diff view settings
Collapse file
Copy file name to clipboardExpand all lines: .github/pull_request_template.md +8-4Lines changed: 8 additions & 4 deletions - Display the source diff
- Display the rich diff
| Original file line number | Diff line number | Diff line change |
|---|
@@ -6,9 +6,13 @@ _Please fill this section with a description what the pull request is trying to |
6 | 6 |
|
7 | 7 | _Please write down a short description of what changes were made._ |
8 | 8 |
|
9 | | -### Notes |
| 9 | +### Testing |
10 | 10 |
|
11 | | -_Please write down any additional notes, remove the section if not applicable._ |
| 11 | +_Please describe the testing already done by you and what testing you request/recommend QA to execute. If you used or created any testing project please link them here too for QA._ |
| 12 | + |
| 13 | +### Risk |
| 14 | + |
| 15 | +_Please describe the potential risks of your changes for the reviewers._ |
12 | 16 |
|
13 | 17 | ### Checklist |
14 | 18 |
|
@@ -17,8 +21,8 @@ Before review: |
17 | 21 | - [ ] Changelog entry added. |
18 | 22 | - Explains the change in `Changed`, `Fixed`, `Added` sections. |
19 | 23 | - For API change contains an example snippet and/or migration example. |
20 | | - - FogBugz ticket attached, example `([case %number%](https://issuetracker.unity3d.com/issues/...))`. |
21 | | - - FogBugz is marked as "Resolved" with *next* release version correctly set. |
| 24 | + - JIRA ticket linked, example ([case %<ID>%](https://issuetracker.unity3d.com/product/unity/issues/guid/<ID>)). If it is a private issue, just add the case ID without a link. |
| 25 | + - Jira port for the next release set as "Resolved". |
22 | 26 | - [ ] Tests added/changed, if applicable. |
23 | 27 | - Functional tests `Area_CanDoX`, `Area_CanDoX_EvenIfYIsTheCase`, `Area_WhenIDoX_AndYHappens_ThisIsTheResult`. |
24 | 28 | - Performance tests. |
|