Skip to content

Commit 60757de

Browse files
authored
Merge pull request #38 from CMSgov/jimfuqian/BB2-fix-pr-template-security-sec-per-new-norm
jimfuqian/BB2 fix PR template security section per new norm
2 parents 2cc1e8b + 2b311a5 commit 60757de

1 file changed

Lines changed: 6 additions & 6 deletions

File tree

.github/pull_request_template.md

Lines changed: 6 additions & 6 deletions
Original file line numberDiff line numberDiff line change
@@ -48,12 +48,12 @@ If you're reviewing this PR, please check these things, in particular:
4848

4949
Submitters should complete the following questionnaire:
5050

51-
- If the answer to any of the questions below is **Yes**, then here's a link to the associated Security Impact Assessment (SIA), security checklist, or other similar document in Confluence: N/A.
52-
- Does this PR add any new software dependencies? **Yes** or **No**.
53-
- Does this PR modify or invalidate any of our security controls? **Yes** or **No**.
54-
- Does this PR store or transmit data that was not stored or transmitted before? **Yes** or **No**.
55-
- If the answer to any of the questions below is **Yes**, then please add StewGoin as a reviewer, and note that this PR should not be merged unless/until he also approves it.
56-
- Do you think this PR requires additional review of its security implications for other reasons? **Yes** or **No**.
51+
* If the answer to any of the questions below is **Yes**, then here's a link to the associated Security Impact Assessment (SIA), security checklist, or other similar document in Confluence: N/A.
52+
* Does this PR add any new software dependencies? **Yes** or **No**.
53+
* Does this PR modify or invalidate any of our security controls? **Yes** or **No**.
54+
* Does this PR store or transmit data that was not stored or transmitted before? **Yes** or **No**.
55+
* If the answer to any of the questions below is **Yes**, then please add a Security Engineer and ISSO as a reviewer, and note that this PR should not be merged unless/until he also approves it.
56+
* Do you think this PR requires additional review of its security implications for other reasons? **Yes** or **No**.
5757

5858
### What Needs to Be Merged and Deployed Before this PR?
5959

0 commit comments

Comments
 (0)