Skip to content

Commit

Permalink
Merge pull request #936 from WildMeOrg/NFRupdate
Browse files Browse the repository at this point in the history
Update PR template to include NFRs
  • Loading branch information
TanyaStere42 authored Dec 12, 2024
2 parents ba23036 + 370893e commit 5572d12
Showing 1 changed file with 7 additions and 8 deletions.
15 changes: 7 additions & 8 deletions .github/pull_request_template.md
Original file line number Diff line number Diff line change
@@ -1,11 +1,10 @@
[Plain-text summary description of the solution]
[Plain-text summary description of the solution. Include any class updates, new or edited functions, significant changes to UX, etc]

PR fixes #[REPLACE WITH ISSUE NUMBER]

**Changes**
- [bullet list include any model updates]
- [new or edited functions]
- [significant changes to UX]
- [design decisions, especially if architectural patterns are new/altered]
- [can use text, screenshots, or files]
- [a callout of any ##TODO or ##FIXMEs found/created as part of the fix]
**Before you Submit!**
* Is all the text internationalized?
* If you made a change to the header, did you update the react, jsp, and html?
* Are all depedencies at a locked version?
* Did you adhere to [best practices](https://wildbook.docs.wildme.org/contribute/code-guide.html)?
* Is there a quick [unit test](https://wildbook.docs.wildme.org/contribute/tests.html) you can add?

0 comments on commit 5572d12

Please sign in to comment.