Skip to content

Commit 04d1540

Browse files
committed
Part nasa/cFS#1079, Adding specialty PR templates and links to the specialty PR templates in the default PR template.
1 parent c44fc67 commit 04d1540

3 files changed

Lines changed: 167 additions & 0 deletions

File tree

Lines changed: 50 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -0,0 +1,50 @@
1+
---
2+
name: Comment / Documentation Change
3+
about: Changes limited to comments, README, docstrings, or other documentation
4+
labels: documentation
5+
---
6+
7+
## Description of Change
8+
9+
<!-- Briefly describe what documentation/comments were changed and why. -->
10+
11+
## Linked Issue
12+
13+
<!-- Required. Reference the GitHub issue this PR addresses, e.g., Closes #123 -->
14+
Closes #
15+
16+
## Areas of Expertise Touched
17+
18+
<!-- Check all that apply so the appropriate Experts can be tagged as reviewers. -->
19+
- [ ] ASTRO
20+
- [ ] CI/CD
21+
- [ ] COSMOS
22+
- [ ] Cybersecurity
23+
- [ ] Docker
24+
- [ ] EDS
25+
- [ ] Git
26+
- [ ] PSPs
27+
- [ ] SBN
28+
- [ ] SMP
29+
- [ ] Tables
30+
- [ ] TSN
31+
- [ ] Unit Tests
32+
- [ ] Other
33+
34+
---
35+
36+
## Author Checklist
37+
38+
- [ ] Linked GitHub issue is referenced above/linked to this PR
39+
- [ ] Documentation generation workflow ran successfully on this branch
40+
- [ ] Changes are limited to documentation/comments (no code behavior changes)
41+
42+
---
43+
44+
## Reviewer Checklist
45+
46+
- [ ] Documentation is clear, accurate, and complete
47+
- [ ] Spelling and grammar reviewed
48+
- [ ] Any links in the documentation have been verified to work
49+
- [ ] Documentation generation workflow output looks correct
50+
- [ ] Appropriate Expert areas have been reviewed
Lines changed: 67 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -0,0 +1,67 @@
1+
---
2+
name: Tools Repository Change
3+
about: Changes to ground-side tools
4+
labels: tools
5+
---
6+
7+
## Description of Change
8+
9+
<!-- Describe what was changed and why. -->
10+
11+
## Linked Issue
12+
13+
Closes #
14+
15+
## Testing Evidence
16+
17+
<!-- Paste relevant test output, link to CI runs, or attach screenshots demonstrating the tool works as intended. -->
18+
19+
## Areas of Expertise Touched
20+
21+
- [ ] ASTRO
22+
- [ ] CI/CD
23+
- [ ] COSMOS
24+
- [ ] Cybersecurity
25+
- [ ] Docker
26+
- [ ] EDS
27+
- [ ] Git
28+
- [ ] PSPs
29+
- [ ] SBN
30+
- [ ] SMP
31+
- [ ] Tables
32+
- [ ] TSN
33+
- [ ] Unit Tests
34+
- [ ] Other
35+
36+
---
37+
38+
## Author Checklist
39+
40+
- [ ] Linked GitHub issue is referenced above
41+
- [ ] Tool has been tested locally and works as intended
42+
- [ ] Tests updated/added to cover the changes
43+
- [ ] All CI workflows ran and passed
44+
- [ ] Testing evidence is included above
45+
- [ ] Self-review of the diff completed
46+
47+
---
48+
49+
## Reviewer Checklist
50+
51+
- [ ] Code logic is correct and matches the stated intent
52+
- [ ] Code is readable, maintainable, and follows project conventions
53+
- [ ] Tests are meaningful and adequately cover the changes
54+
- [ ] Testing evidence reviewed and acceptable
55+
- [ ] **The change has been exercised by the tests** (not just that tests pass — the new/changed code paths are actually covered)
56+
- [ ] **Reviewer has independently verified the change behaves as described** (e.g., by running the tool locally, reviewing CI output in detail, or performing additional ad-hoc testing as warranted)
57+
- [ ] Error handling is appropriate
58+
- [ ] Appropriate Expert areas have been reviewed
59+
60+
### Reviewer Testing Notes
61+
62+
<!--
63+
Describe how you independently verified this change. For example:
64+
- "Pulled the branch locally, ran the tool against [test input], output matched expectations."
65+
- "Reviewed CI run #1234; verified the new code paths are covered."
66+
- "Performed ad-hoc test by [describe scenario]; observed [result]."
67+
-->
Lines changed: 50 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -0,0 +1,50 @@
1+
---
2+
name: Workflows Change
3+
about: Changes to files under .github/workflows/
4+
labels: ci/workflows
5+
---
6+
7+
## Description of Change
8+
9+
<!-- Describe what workflow(s) were changed and why. -->
10+
11+
## Linked Issue
12+
13+
Closes #
14+
15+
## Areas of Expertise Touched
16+
17+
- [ ] ASTRO
18+
- [ ] CI/CD
19+
- [ ] COSMOS
20+
- [ ] Cybersecurity
21+
- [ ] Docker
22+
- [ ] EDS
23+
- [ ] Git
24+
- [ ] PSPs
25+
- [ ] SBN
26+
- [ ] SMP
27+
- [ ] Tables
28+
- [ ] TSN
29+
- [ ] Unit Tests
30+
- [ ] Other
31+
32+
---
33+
34+
## Author Checklist
35+
36+
- [ ] Linked GitHub issue is referenced above
37+
- [ ] Workflow changes were tested on a branch/fork before submitting this PR
38+
- [ ] All workflows ran successfully on this branch
39+
40+
---
41+
42+
## Reviewer Checklist
43+
44+
- [ ] Workflow logic is correct and achieves the stated goal
45+
- [ ] Third-party actions are from trusted sources and pinned to a specific SHA/tag
46+
- [ ] Secrets are handled appropriately (no secrets exposed in logs, least-privilege usage)
47+
- [ ] Workflow permissions follow the principle of least privilege
48+
- [ ] Changes do not break or bypass existing branch protection rules
49+
- [ ] All workflow runs on this PR completed successfully
50+
- [ ] Appropriate Expert areas have been reviewed

0 commit comments

Comments
 (0)