-
Notifications
You must be signed in to change notification settings - Fork 2
Commit
This commit does not belong to any branch on this repository, and may belong to a fork outside of the repository.
misc: Improve the PR template with comments
- Loading branch information
1 parent
3e83422
commit 0ce7ccf
Showing
1 changed file
with
18 additions
and
17 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,27 +1,28 @@ | ||
Related to # | ||
## Related issue(s) | ||
|
||
Changes proposed in this pull request: | ||
<!-- Copy-paste the URL to the related issue(s) if any ("N/A" if not applicable). --> | ||
|
||
- | ||
- | ||
- | ||
## Changes | ||
|
||
How to test the feature manually: | ||
<!-- List the changes you’ve made in this pull request in order to help the | ||
-- reviewers to understand how to review it. --> | ||
|
||
1. | ||
2. | ||
3. | ||
## How to test manually | ||
|
||
Pull request checklist: | ||
<!-- List actions (step by step) of what have to be done in order to test your | ||
-- changes manually ("N/A" if not applicable). --> | ||
|
||
## Checklist | ||
|
||
<!-- Make sure all the todos are checked before asking for review. If you think | ||
-- one of the item isn’t applicable to this PR, please check it anyway and | ||
-- precise "N/A" next to it. --> | ||
|
||
- [ ] code is manually tested | ||
- [ ] authorizations are checked | ||
- [ ] permissions / authorizations are verified | ||
- [ ] interface works on both mobiles and big screens | ||
- [ ] accessibility has been tested | ||
- [ ] tests are updated | ||
- [ ] tests are up-to-date | ||
- [ ] locales are synchronized | ||
- [ ] copyright notice is updated | ||
- [ ] documentation is updated (including comments, commit messages, migration notes…) | ||
|
||
_If you think one of the item isn’t applicable to the PR, please check it | ||
anyway and precise `N/A` next to it._ | ||
- [ ] copyright notices are up-to-date | ||
- [ ] documentation is up-to-date (including migration notes) |