|
1 | | -## Related Issue(required) |
| 1 | +**Note**: Confirm our [contribution guidelines](https://autowarefoundation.github.io/autoware-documentation/main/contributing/) before submitting a pull request. |
2 | 2 |
|
3 | | -<!-- Link related issue --> |
| 3 | +Click the `Preview` tab and select a PR template: |
4 | 4 |
|
5 | | -## Description(required) |
6 | | - |
7 | | -<!-- Describe what this PR changes. --> |
8 | | - |
9 | | -## Review Procedure(required) |
10 | | - |
11 | | -<!-- Explain how to review this PR. --> |
12 | | - |
13 | | -## Related PR(optional) |
14 | | - |
15 | | -<!-- Link related PR --> |
16 | | - |
17 | | -## Pre-Review Checklist for the PR Author |
18 | | - |
19 | | -**PR Author should check the checkboxes below when creating the PR.** |
20 | | - |
21 | | -- [ ] Read [commit-guidelines][commit-guidelines] |
22 | | -- [ ] Assign PR to reviewer |
23 | | - |
24 | | -If you are adding new package following items are required: |
25 | | - |
26 | | -- [ ] Documentation with description of the package is available |
27 | | -- [ ] A sample launch file and parameter file are available if the package contains executable nodes |
28 | | - |
29 | | -## Checklist for the PR Reviewer |
30 | | - |
31 | | -**Reviewers should check the checkboxes below before approval.** |
32 | | - |
33 | | -- [ ] Commits are properly organized and messages are according to the guideline |
34 | | -- [ ] PR title describes the changes |
35 | | - |
36 | | -## Post-Review Checklist for the PR Author |
37 | | - |
38 | | -**PR Author should check the checkboxes below before merging.** |
39 | | - |
40 | | -- [ ] All open points are addressed and tracked via issues or tickets |
41 | | - |
42 | | -## CI Checks |
43 | | - |
44 | | -- **Build and test for PR / build-and-test-pr**: Required to pass before the merge. |
45 | | -- **Build and test for PR / clang-tidy-pr**: NOT required to pass before the merge. It is up to the reviewer(s). Found false positives? See the [guidelines][clang-tidy-guidelines]. |
46 | | -- **Check spelling**: NOT required to pass before the merge. It is up to the reviewer(s). See [here][spell-check-dict] if you want to add some words to the spell check dictionary. |
47 | | - |
48 | | -[commit-guidelines]: https://www.conventionalcommits.org/en/v1.0.0/ |
49 | | -[spell-check-dict]: https://github.com/tier4/autoware-spell-check-dict#how-to-contribute |
| 5 | +- [Standard change](?expand=1&template=standard-change.md) |
| 6 | +- [Small change](?expand=1&template=small-change.md) |
0 commit comments