|
1 |
| -## What does this MR do? |
| 1 | +## What does this merge request do? |
2 | 2 |
|
3 |
| -Addresses #NUMBER+ |
| 3 | +<!-- TODO Link associated issue from title, like: <title> #NUMBER --> |
4 | 4 |
|
5 |
| -<!-- Briefly describe what this MR is about. --> |
| 5 | +<!-- TODO Briefly list what this merge request is about --> |
6 | 6 |
|
7 | 7 | ## Author's checklist
|
8 | 8 |
|
9 |
| -- [ ] The MR fully addresses the requirements of the associated task. |
10 |
| -- [ ] I did a self-review of the changes and did not spot any issues. Among others, this includes: |
11 |
| - * I added unit tests for new/changed behavior; all test pass. |
12 |
| - * My code conforms to our coding standards and guidelines. |
13 |
| - * My changes are prepared in a way that makes the review straightforward for the reviewer. |
14 |
| -- [ ] I assigned a reviewer and added the Review label. |
| 9 | +- [ ] This merge request fully addresses the requirements of the associated task |
| 10 | +- [ ] I did a self-review of the changes and did not spot any issues, among others: |
| 11 | + - I added unit tests for new or changed behavior; existing and new tests pass |
| 12 | + - My code conforms to our coding standards and guidelines |
| 13 | + - My changes are prepared (focused commits, good messages) so reviewing them is easy for the reviewer |
| 14 | +- [ ] I assigned a reviewer to request review |
15 | 15 |
|
16 |
| -## Review checklist |
| 16 | +## Reviewer's checklist |
17 | 17 |
|
18 |
| -- [ ] I reviewed all changes line-by-line and addressed relevant issues |
| 18 | +- [ ] I reviewed all changes line-by-line and addressed relevant issues |
19 | 19 | - [ ] The requirements of the associated task are fully met
|
20 |
| -- [ ] I can confirm that: |
21 |
| - * CI passes |
22 |
| - * Coverage percentages do not decrease |
23 |
| - * New code conforms to standards and guidelines |
24 |
| - * If applicable, additional checks were done for special code changes (e.g. core performance, binary size, OSS licenses) |
| 20 | +- [ ] I can confirm that: |
| 21 | + - CI passes |
| 22 | + - If applicable, coverage percentages do not decrease |
| 23 | + - New code conforms to standards and guidelines |
| 24 | + - If applicable, additional checks were done for special code changes (e.g. core performance, binary size, OSS licenses) |
0 commit comments