Skip to content

build: PR Template Updates#3033

Draft
scotttjob wants to merge 1 commit intomasterfrom
scott_t/pr-template
Draft

build: PR Template Updates#3033
scotttjob wants to merge 1 commit intomasterfrom
scott_t/pr-template

Conversation

@scotttjob
Copy link
Copy Markdown
Contributor

Motivations

  1. Start a discussion about our PR Template + Process. This is a draft, it is by no means final.

Removed all the sections in favour of 4 new sections:

Motivations == Why Is This Changing?
Changes == What Is Changing?

Remaining sections reduced to:

Consumer Impact
Validation
Reviewer Notes

The goal is to track intent, outcome, impact and risk in a reduced set of fields.

Our Consumer Impact section will be interesting to see human filled versus danger-rule/deterministic check. It may highlight gaps in the danger-rule process, which is what the intention here is, to improve that danger rule over time.

Changes

  1. The PR Template

Changed

Deprecated

Removed

Fixed

Security

Testing

Changes can be
tested via Pre-release


In Atlantis we use Github's built in pull request reviews.

Copy link
Copy Markdown
Contributor

@jdeichert jdeichert left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Leaving my stamp of approval (in draft mode) 👍

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Development

Successfully merging this pull request may close these issues.

2 participants