|
1 |
| -# Pull Request Template |
| 1 | +# Pull Request |
2 | 2 |
|
3 |
| -## Summary |
| 3 | +## Description |
4 | 4 |
|
5 |
| -<!-- Provide a concise description of your changes. What does this PR do? --> |
| 5 | +Please include a summary of the change and which issue is fixed. Also include relevant motivation and context. |
6 | 6 |
|
7 |
| -## Motivation & Context |
| 7 | +Fixes # (issue) |
8 | 8 |
|
9 |
| -<!-- Why is this change necessary? What problem does it solve? Reference any related issues. --> |
| 9 | +## Type of change |
10 | 10 |
|
11 |
| -## Related Issues |
12 |
| - |
13 |
| -<!-- List any related issues, e.g., Fixes #123 or Closes #456. --> |
14 |
| - |
15 |
| -## Implementation Details |
16 |
| - |
17 |
| -<!-- Briefly describe your approach, design decisions, and any patterns or principles followed. --> |
| 11 | +- [ ] Bug fix |
| 12 | +- [ ] New feature |
| 13 | +- [ ] Breaking change |
| 14 | +- [ ] Documentation update |
| 15 | +- [ ] Other (please describe): |
18 | 16 |
|
19 | 17 | ## How Has This Been Tested?
|
20 | 18 |
|
21 |
| -<!-- Describe the testing strategy. List manual steps, automated tests, and edge cases considered. --> |
| 19 | +Please describe the tests that you ran to verify your changes. Provide instructions so we can reproduce. Please also list any relevant details for your test configuration. |
22 | 20 |
|
23 | 21 | ## Checklist
|
24 | 22 |
|
25 |
| -- [ ] My code follows the project's code style and conventions |
26 |
| -- [ ] I have added/updated relevant documentation |
27 |
| -- [ ] I have added/updated tests to cover my changes |
28 |
| -- [ ] All existing and new tests pass locally |
29 |
| -- [ ] I have considered security and data validation aspects |
| 23 | +- [ ] My code follows the style guidelines of this project |
| 24 | +- [ ] I have performed a self-review of my own code |
| 25 | +- [ ] I have commented my code, particularly in hard-to-understand areas |
| 26 | +- [ ] I have made corresponding changes to the documentation |
| 27 | +- [ ] My changes generate no new warnings |
| 28 | +- [ ] I have added tests that prove my fix is effective or that my feature works |
| 29 | +- [ ] New and existing unit tests pass locally with my changes |
| 30 | +- [ ] Any dependent changes have been merged and published in downstream modules |
30 | 31 |
|
31 |
| -## Notes for Reviewers |
| 32 | +## Additional context |
32 | 33 |
|
33 |
| -<!-- Add any extra context, open questions, or areas where you'd like feedback. --> |
| 34 | +Add any other context or screenshots about the pull request here. |
0 commit comments