Skip to content

PULL_REQUEST_TEMPLATE.md

Latest commit

 

History

History
38 lines (21 loc) · 1.48 KB

PULL_REQUEST_TEMPLATE.md

File metadata and controls

38 lines (21 loc) · 1.48 KB

Description

Required: Please provide a brief description of what this pull request is trying to accomplish.

Context, Consequences, & Considerations

Required: Please step through the following list, pausing at each item to consider your change in relation to the item's context. Check the box to mark that it applies, and enter your relevant notes under the item.

  • Security: This has security implications. This includes (but not limited to) adding users, modifying user/app permissions, network rules/policies, changing a system interconnection, or changing an authorization strategy.
    • This PR does not require security review. These changes are part of a project plan that has already undergone security review. The link is provided below.
    • This PR requires security review. Add the security label to this PR then request a review from the Security Code Reviewers Team.
  • Execution: This change requires commands to be run outside of the normal merge.
  • Impact: This change may cause service interruptions.
  • Testing: How did you test this change (unit tests, acceptance tests, etc.)? Did you do any manual testing?
  • Testing: How will you confirm this change once it's merged?
  • Documentation: Documentation to reflect this change has been added to Confluence or Zendesk.
  • All items of the checklist have been considered and this PR description is complete.