diff --git a/.github/PULL_REQUEST_TEMPLATE/comment_documentation_change.md b/.github/PULL_REQUEST_TEMPLATE/comment_documentation_change.md index 4c5537c..de1252a 100644 --- a/.github/PULL_REQUEST_TEMPLATE/comment_documentation_change.md +++ b/.github/PULL_REQUEST_TEMPLATE/comment_documentation_change.md @@ -16,6 +16,7 @@ Closes # ## Areas of Expertise Touched + - [ ] ASTRO - [ ] CI/CD - [ ] COSMOS diff --git a/.github/PULL_REQUEST_TEMPLATE/fsw_code_change.md b/.github/PULL_REQUEST_TEMPLATE/fsw_code_change.md index 4701113..2a8f865 100644 --- a/.github/PULL_REQUEST_TEMPLATE/fsw_code_change.md +++ b/.github/PULL_REQUEST_TEMPLATE/fsw_code_change.md @@ -31,6 +31,8 @@ Closes # ## Areas of Expertise Touched + + - [ ] ASTRO - [ ] CI/CD - [ ] COSMOS @@ -65,10 +67,9 @@ Closes # ## Reviewer Checklist - [ ] Code logic is correct and matches the stated intent -- [ ] Code is readable, maintainable, and follows project conventions +- [ ] Code is readable, maintainable, and follows project conventions (ask your lead if you are unsure of where to find these conventions) - [ ] `.clang-format` has been applied - [ ] Static analysis results reviewed and acceptable -- [ ] Unit tests are meaningful and adequately cover the changes - [ ] **The change has been exercised by the unit tests** (not just that tests pass — the new/changed code paths are actually covered) - [ ] **COSMOS test suite was executed against this change** and results reviewed (or confirmed N/A with justification) - [ ] **Reviewer has independently verified the change behaves as described** (e.g., by running the tests locally, reviewing CI output in detail, or performing additional ad-hoc testing as warranted) diff --git a/.github/PULL_REQUEST_TEMPLATE/workflows_change.md b/.github/PULL_REQUEST_TEMPLATE/workflows_change.md index 8fe91cb..700e398 100644 --- a/.github/PULL_REQUEST_TEMPLATE/workflows_change.md +++ b/.github/PULL_REQUEST_TEMPLATE/workflows_change.md @@ -14,6 +14,8 @@ Closes # ## Areas of Expertise Touched + + - [ ] ASTRO - [ ] CI/CD - [ ] COSMOS