mirror of
https://github.com/SeedSigner/seedsigner.git
synced 2026-10-05 15:08:25 +00:00
Update PR template
This commit is contained in:
@@ -1,8 +1,16 @@
|
||||
## Description
|
||||
|
||||
_Describe the change simply. Provide a reason for the change._
|
||||
<i>
|
||||
|
||||
_Include screenshots of any new or modified screens (or at least explain why they were omitted)_
|
||||
* 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?
|
||||
|
||||
</i>
|
||||
|
||||
---
|
||||
|
||||
This pull request is categorized as a:
|
||||
|
||||
@@ -12,21 +20,45 @@ This pull request is categorized as a:
|
||||
- [ ] Documentation
|
||||
- [ ] Other
|
||||
|
||||
---
|
||||
|
||||
## Checklist
|
||||
|
||||
- [ ] I’ve run `pytest` and made sure all unit tests pass before submitting the PR
|
||||
|
||||
If you modified or added functionality/workflow, did you add new unit tests?
|
||||
|
||||
- [ ] No, I’m a fool
|
||||
- [ ] Yes
|
||||
<b>I ran `pytest` locally</b>
|
||||
- [ ] All tests passed before submitting the PR
|
||||
- [ ] I couldn't run the tests
|
||||
- [ ] N/A
|
||||
|
||||
I have tested this PR on the following platforms/os:
|
||||
|
||||
<b>I included screenshots of any new or modified screens</b>
|
||||
Should be part of the PR description above.
|
||||
- [ ] Yes
|
||||
- [ ] No (explain why they were omitted)
|
||||
- [ ] N/A
|
||||
|
||||
|
||||
<b>I added or updated tests</b>
|
||||
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
|
||||
|
||||
|
||||
<b>I tested this PR hands-on on the following platform(s):</b>
|
||||
- [ ] Raspberry Pi OS [Manual Build](https://github.com/SeedSigner/seedsigner/blob/dev/docs/manual_installation.md)
|
||||
- [ ] [SeedSigner OS](https://github.com/SeedSigner/seedsigner-os) on a Pi0/Pi0W board
|
||||
- [ ] Other
|
||||
- [ ] Emulator
|
||||
|
||||
|
||||
Note: 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 to review, test, and merge.
|
||||
<b>I have reviewed these notes:</b>
|
||||
* 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.
|
||||
* We appreciate your efforts, but we're a small team of volunteers so PR review can be a very slow process.
|
||||
* Please only "@" mention a contributor if their input is truly needed to enable further progress.
|
||||
|
||||
- [ ] I understand
|
||||
|
||||
---
|
||||
|
||||
Thank you! Please join our [Devs' Telegram group](https://t.me/seedsigner_new_devs) to get more involved.
|
||||
Reference in New Issue
Block a user