From b65d8fae5aaa42d276a92db434543097154f12b3 Mon Sep 17 00:00:00 2001 From: Rachel Moser <33227096+rlmoser99@users.noreply.github.com> Date: Fri, 21 Oct 2022 21:31:30 -0500 Subject: [PATCH] PR Template: Flip order of checklist and details (#231) * PR Template: Flip order of checklist and details Because: If the details are first, they are viewable in Discord's webhook. This PR: * Moves the detail sections to the beginning of the template * Moves the checklist section to the end of the template * Adds an issue section to link the issue * Adds the Because, This PR, and Issue sections to checklist * Reduce wording that is not essential --- .github/PULL_REQUEST_TEMPLATE.md | 42 +++++++++++++++++--------------- 1 file changed, 22 insertions(+), 20 deletions(-) diff --git a/.github/PULL_REQUEST_TEMPLATE.md b/.github/PULL_REQUEST_TEMPLATE.md index 3a7b485fd2f..03cf1baeaca 100644 --- a/.github/PULL_REQUEST_TEMPLATE.md +++ b/.github/PULL_REQUEST_TEMPLATE.md @@ -1,30 +1,32 @@ - + -Complete the following REQUIRED checkboxes: - -- [ ] I have thoroughly read and understand [The Odin Project Contributing Guide](https://github.com/TheOdinProject/theodinproject/blob/main/CONTRIBUTING.md) -- [ ] The title of this PR follows the `location of change: brief description of change` format, e.g. `01-flex-center: Update self check` +## Because + -Complete the following checkbox ONLY IF it is applicable to your PR. You can complete it later if it is not currently applicable: -- [ ] I have ensured that the TOP solution files match the Desired Outcome image -
+## This PR + -**1. Because:** - -Closes #XXXXX +## Issue + +If this PR does not close, but is related to another issue or PR, you can link it as above without the 'Closes' keyword, e.g. 'Related to #2013'. +--> +Closes #XXXXX +## Additional Information + -**3. Additional Information:** - +## Pull Request Requirements + +- [ ] I have thoroughly read and understand [The Odin Project Contributing Guide](https://github.com/TheOdinProject/theodinproject/blob/main/CONTRIBUTING.md) +- [ ] The title of this PR follows the `location of change: brief description of change` format, e.g. `01-flex-center: Update self check` +- [ ] The `Because` section summarizes the reason for this PR +- [ ] The `This PR` section has a bullet point list describing the changes in this PR +- [ ] If this PR addresses an open issue, it is linked in the `Issue` section +- [ ] If applicable, I have ensured that the TOP solution files match the Desired Outcome image