What changed, and why it matters
This commit only updates the GitHub pull request template, which is a documentation/process file. It contains no firmware code changes and no security-relevant modifications.
Recommended action
No action required; this is a routine documentation update.
Security signals we found
No strong security signals were identified.
Evidence from the diff
The diff modifies .github/pull_request_template.md, reorganizing instructions for core developers and external contributors. No source code, build scripts, cryptographic logic, or security controls are changed.
Changed components
.github/pull_request_template.mdInspect captured patch +15 / −7
diff --git a/.github/pull_request_template.md b/.github/pull_request_template.md
index 0f92eae2..51130452 100644
--- a/.github/pull_request_template.md
+++ b/.github/pull_request_template.md
@@ -1,12 +1,20 @@
<!--
+For external contributors:
+- Please open an issue before submitting a PR so we can discuss whether we want to proceed with it.
+
For core developers:
+1. Initial PR setup
- Assign yourself to the PR.
-- Set the priority to match the original issue.
-- Add the PR to the current sprint.
-- If it's a draft PR, mark it as "In Progress."
-- If it's a final PR, mark it as "Needs Review."
-- If needed, add a `Notes for QA` section.
+- Add it to the "Firmware" project
+ - Set Priority (match issue priority if it exists)
+ - Set Team
+ - Set Sprint (target release)
-For external contributors:
-- Please open an issue before submitting a PR so we can discuss whether we want to proceed with it.
+2. Development status
+- Draft PRs: Set status to "In Progress"
+- Final PRs: Set status to "Needs Review"
+
+3. Post-merge status
+- Testable: Set status to "Needs QA" and add a `Notes for QA` section with clear instructions on how to test the functionality.
+- Not Testable: Set status to "Done (no QA)"
-->
Risk score
Our methodology →Why this scored 15/100
Human-validated context
Community notes
Notes can correct, qualify, or add evidence to the AI analysis. Every note shown here has been validated by a human moderator.
No validated notes yet.
The AI analysis stands alone for now. Submit a note if you can add evidence or important context.