|
1 | | -## π Description |
| 1 | +## Task Issue |
2 | 2 |
|
3 | | -<!-- Briefly describe the changes in this PR --> |
| 3 | +Closes #<!-- issue number --> |
4 | 4 |
|
5 | | -## π Related Issues |
| 5 | +## Description |
6 | 6 |
|
7 | | -<!-- Link to related issues: Fixes #123, Closes #456 --> |
| 7 | +<!-- What changes does this PR make and why? --> |
8 | 8 |
|
9 | | -## π§ͺ Type of Change |
| 9 | +## Checklist |
10 | 10 |
|
11 | | -- [ ] π Bug fix (non-breaking change which fixes an issue) |
12 | | -- [ ] β¨ New feature (non-breaking change which adds functionality) |
13 | | -- [ ] π₯ Breaking change (fix or feature that would cause existing functionality to not work as expected) |
14 | | -- [ ] π Documentation update |
15 | | -- [ ] π§ Maintenance / refactoring |
16 | | -- [ ] π Security fix |
| 11 | +- [ ] Linked to task issue |
| 12 | +- [ ] Changes are minimal (only what the task requires) |
| 13 | +- [ ] Tests added/updated |
| 14 | +- [ ] All tests pass (`make test`) |
| 15 | +- [ ] Linting passes (`make lint`) |
| 16 | +- [ ] No secrets or credentials committed |
| 17 | +- [ ] Documentation updated (if applicable) |
| 18 | +- [ ] Follows project conventions (docs/conventions.md) |
17 | 19 |
|
18 | | -## β
Checklist |
| 20 | +## Reviewer Notes |
19 | 21 |
|
20 | | -- [ ] My code follows the project's style guidelines |
21 | | -- [ ] I have performed a self-review of my code |
22 | | -- [ ] I have commented my code, particularly in hard-to-understand areas |
23 | | -- [ ] I have made corresponding changes to the documentation |
24 | | -- [ ] My changes generate no new warnings |
25 | | -- [ ] I have added tests that prove my fix is effective or that my feature works |
26 | | -- [ ] New and existing unit tests pass locally with my changes |
| 22 | +<!-- Anything reviewers should know or pay attention to --> |
27 | 23 |
|
28 | | -## πΈ Screenshots (if applicable) |
| 24 | +## Security Considerations |
29 | 25 |
|
30 | | -<!-- Add screenshots to help explain your changes --> |
31 | | - |
32 | | -## π Additional Notes |
33 | | - |
34 | | -<!-- Any additional information that reviewers should know --> |
| 26 | +<!-- Any security implications of these changes? (new inputs, auth changes, dependency updates, etc.) --> |
0 commit comments