From aa4e3293a4b1807872e57855163348d341c5b2ab Mon Sep 17 00:00:00 2001 From: kdmukai <934746+kdmukai@users.noreply.github.com> Date: Fri, 13 Feb 2026 19:57:23 -0600 Subject: [PATCH] Explicit description template; minor presentation reformatting; docs link fix --- .github/pull_request_template.md | 51 +++++++++++++++++++++++--------- 1 file changed, 37 insertions(+), 14 deletions(-) diff --git a/.github/pull_request_template.md b/.github/pull_request_template.md index b3f4e0fa..13c1912a 100644 --- a/.github/pull_request_template.md +++ b/.github/pull_request_template.md @@ -1,14 +1,31 @@ ## Description - +### Problem or Issue being addressed -* What is the problem or issue you're addressing here? - * Include any relevant background info, links to resources, BIPs, etc. -* What is your solution? - * Include key tech details to help reviewers understand your changes and rationale. -* Any tradeoffs to discuss, follow-up steps, etc? + - +### Solution + + + +### Additional Information + + + +### Screenshots + + --- @@ -24,33 +41,39 @@ This pull request is categorized as a: ## Checklist -I ran `pytest` locally +#### I ran `pytest` locally - [ ] All tests passed before submitting the PR - [ ] I couldn't run the tests - [ ] N/A +--- + +#### I included screenshots of any new or modified screens -I included screenshots of any new or modified screens Should be part of the PR description above. - [ ] Yes -- [ ] No (explain why they were omitted) +- [ ] No - [ ] N/A +--- + +#### I added or updated tests -I added or updated tests Any new or altered functionality should be covered in a unit test. Any new or updated sequences require FlowTests. - [ ] Yes - [ ] No, I’m a fool - [ ] N/A +--- -I tested this PR hands-on on the following platform(s): -- [ ] Raspberry Pi OS [Manual Build](https://github.com/SeedSigner/seedsigner/blob/dev/docs/manual_installation.md) +#### I tested this PR hands-on on the following platform(s): +- [ ] Raspberry Pi OS [Manual Build](https://github.com/SeedSigner/seedsigner/blob/dev/docs/raspberry_pi_os_build_instructions.md) - [ ] [SeedSigner OS](https://github.com/SeedSigner/seedsigner-os) on a Pi0/Pi0W board - [ ] Emulator +--- -I have reviewed these notes: +#### I have reviewed these notes: * Keep your changes limited in scope. * If you uncover other issues or improvements along the way, ideally submit those as a separate PR. * The more complicated the PR, the harder it is to review, test, and merge.