Skip to content

Commit 1f0537a

Browse files
authored
Update PR template (#1235)
1 parent 5bbca90 commit 1f0537a

File tree

1 file changed

+10
-19
lines changed

1 file changed

+10
-19
lines changed

.github/pull_request_template.md

Lines changed: 10 additions & 19 deletions
Original file line numberDiff line numberDiff line change
@@ -7,24 +7,14 @@ This PR template is here to help ensure you're setup for success:
77
**JIRA Ticket:**
88
[BB2-XXXX](https://jira.cms.gov/browse/BB2-XXXX)
99

10-
1110
### What Does This PR Do?
11+
1212
<!--
1313
Add detailed description & discussion of changes here.
14-
The contents of this section should be used as your commit message (unless you merge the PR via a merge commit, of course).
15-
16-
Please follow standard Git commit message guidelines:
17-
* First line should be a capitalized, short (50 chars or less) summary.
18-
* The rest of the message should be in standard Markdown format, wrapped to 72 characters.
19-
* Describe your changes in imperative mood, e.g. "make xyzzy do frotz" instead of "[This patch] makes xyzzy do frotz" or "[I] changed xyzzy to do frotz", as if you are giving orders to the codebase to change its behavior.
20-
* List all relevant Jira issue keys, one per line at the end of the message, per: <https://confluence.atlassian.com/jirasoftwarecloud/processing-issues-with-smart-commits-788960027.html>.
21-
22-
Reference: <https://git-scm.com/book/en/v2/Distributed-Git-Contributing-to-a-Project>.
2314
-->
2415

25-
Replace me.
26-
2716
### What Should Reviewers Watch For?
17+
2818
<!--
2919
Common items include:
3020
* Is this likely to address the goals expressed in the user story?
@@ -46,24 +36,25 @@ Have you fully verified and tested these changes? Is the acceptance criteria met
4636

4737
### What Security Implications Does This PR Have?
4838

49-
Please indicate if this PR does any of the following:
39+
Please indicate if this PR does any of the following:
5040

5141
* Adds any new software dependencies
5242
* Modifies any security controls
5343
* Adds new transmission or storage of data
54-
* Any other changes that could possibly affect security?
44+
* Any other changes that could possibly affect security?
5545

56-
* [ ] Yes, one or more of the above security implications apply. This PR must not be merged without the ISSO or team security engineer's approval.
46+
- [ ] Yes, one or more of the above security implications apply. This PR must not be merged without the ISSO or team
47+
security engineer's approval.
5748

58-
5949
### Any Migrations?
6050

6151
<!--
6252
Make sure to work with whoever is doing the deploy so they are aware of any migrations that may need to be run
6353
-->
6454

6555
* [ ] Yes, there are migrations
66-
* [ ] The migrations should be run PRIOR to the code being deployed
67-
* [ ] The migrations should be run AFTER the code is deployed
68-
* [ ] There is a more complicated migration plan (downtime, etc) <!-- Make sure to include the details of the plan below -->
56+
* [ ] The migrations should be run PRIOR to the code being deployed
57+
* [ ] The migrations should be run AFTER the code is deployed
58+
* [ ] There is a more complicated migration plan (downtime,
59+
etc) <!-- Make sure to include the details of the plan below -->
6960
* [ ] No migrations

0 commit comments

Comments
 (0)