PR Template: Flip order of checklist and details (#289)
* 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
This commit is contained in:
parent
0747078d97
commit
a05d4d60c8
1 changed files with 22 additions and 20 deletions
42
.github/PULL_REQUEST_TEMPLATE.md
vendored
42
.github/PULL_REQUEST_TEMPLATE.md
vendored
|
@ -1,30 +1,32 @@
|
|||
<!-- Thank you for taking the time to contribute to The Odin Project. In order to get a pull request (PR) closed in a reasonable amount of time, you must include a baseline of information about the changes you are proposing. Please read this template in its entirety before filling it out to ensure that it is filled out correctly. -->
|
||||
<!-- Thank you for taking the time to contribute to The Odin Project. In order to get this pull request (PR) merged in a reasonable amount of time, you must complete this entire template. -->
|
||||
|
||||
Complete the following REQUIRED checkboxes:
|
||||
<!-- While editing this template, replace the whitespace between the square brackets with an 'x', e.g. [x] -->
|
||||
- [ ] 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_helloWorld: Update test cases`
|
||||
## Because
|
||||
<!-- Summarize the purpose or reasons for this PR, e.g. what problem it solves or what benefit it provides. -->
|
||||
|
||||
Complete the following checkboxes ONLY IF they are applicable to your PR. You can complete them later if they are not currently applicable:
|
||||
- [ ] I have ensured any exercise files included in this PR have passed all of their tests
|
||||
|
||||
<hr>
|
||||
## This PR
|
||||
<!-- A bullet point list of one or more items describing the specific changes. -->
|
||||
|
||||
**1. Because:**
|
||||
|
||||
## Issue
|
||||
<!--
|
||||
If this PR closes an open issue, replace the XXXXX below with the issue number, e.g. Closes #2013. Or if the issue is in another TOP repo replace the #XXXXX with the URL of the issue, e.g. Closes https://github.com/TheOdinProject/curriculum/issues/XXXXX
|
||||
If this PR closes an open issue in this repo, replace the XXXXX below with the issue number, e.g. Closes #2013.
|
||||
|
||||
Otherwise, provide a clear and concise reason for your pull request, e.g. what problem it solves or what benefit it provides. If this PR is related to, but does not close, another issue or PR, you can also link it as above without the 'Closes' keyword, e.g. "Related to #2013".
|
||||
-->
|
||||
If this PR closes an open issue in another TOP repo, replace the #XXXXX with the URL of the issue, e.g. Closes https://github.com/TheOdinProject/curriculum/issues/XXXXX
|
||||
|
||||
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
|
||||
|
||||
|
||||
**2. This PR:**
|
||||
<!--
|
||||
A bullet point list of one or more items outlining what was done in this PR to solve the problem(s) or implement the feature/enhancement.
|
||||
-->
|
||||
## Additional Information
|
||||
<!-- Any other information about this PR, such as a link to a Discord discussion. -->
|
||||
|
||||
|
||||
**3. Additional Information:**
|
||||
<!-- Any additional information about the PR, such as a link to a Discord discussion, etc. -->
|
||||
|
||||
## Pull Request Requirements
|
||||
<!-- Replace the whitespace between the square brackets with an 'x', e.g. [x]. After you create the PR, they will become checkboxes that you can click on. -->
|
||||
- [ ] 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_helloWorld: Update test cases`
|
||||
- [ ] 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 this PR includes changes that needs to be updated on the `solutions` branch, I have created another PR (and linked it to this PR).
|
||||
|
|
Loading…
Reference in a new issue