Aligns PR Template with Apply (#4419)

* Copied updated template from Apply codebase * Removed Trello card header Our Trello integration adds a link to Trello as a comment when the PR is attached to a Trello card * Update .github/PULL_REQUEST_TEMPLATE.md Co-authored-by: avinhurry <[email protected]> * PR changes --------- Co-authored-by: avinhurry <[email protected]>
DFE-Digital · Aug 2, 2024 · 0f55856 · 0f55856
1 parent f0c924c
commit 0f55856
Showing 1 changed file with 17 additions and 6 deletions.
diff --git a/.github/PULL_REQUEST_TEMPLATE.md b/.github/PULL_REQUEST_TEMPLATE.md
@@ -1,14 +1,25 @@
-### Context
+## Context
 
-### Changes proposed in this pull request
+<!-- Why are you making this change? What might surprise someone about it? -->
 
-### Guidance to review
+## Changes proposed in this pull request
 
-### Checklist
+<!-- If there are UI changes, please include Before and After screenshots. -->
 
+## Guidance to review
+
+<!-- How could someone else check this work? Which parts do you want more feedback on? -->
+
+## Things to check
+
+- [ ] This code does not rely on migrations in the same Pull Request
+- [ ] If this code includes a migration adding or changing columns, it also backfills existing records for consistency
+- [ ] If this code adds a column to the DB, decide whether it needs to be in analytics yml file or analytics blocklist
+- [ ] API release notes have been updated if necessary
+- [ ] Required environment variables have been updated added to the Azure KeyVault
+- [ ] Inform data insights team due to database changes
 - [ ] Make sure all information from the Trello card is in here
-- [ ] Attach to Trello card
 - [ ] Rebased main
 - [ ] Cleaned commit history
 - [ ] Tested by running locally
-- [ ] Inform data insights team due to database changes
+- [ ] Attach PR to Trello card