Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Add pull request template #2161

Merged
merged 2 commits into from Sep 17, 2021
Merged

Conversation

emosbaugh
Copy link
Member

This PR includes a PR template to set some guidelines on contributing including release notes, documentation and testing.

If no, just write "NONE" in the release-note block below.
If yes, a release note is required:
-->
```release-note
Copy link
Member Author

Choose a reason for hiding this comment

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

Do we ask for a link to a PR into kots.io? This is a lot to ask of an external contributor.

Copy link
Contributor

Choose a reason for hiding this comment

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

I don't think so. Maybe we can ask that they supply a release note for the feature in the PR request and we can go through merged PRs and grab them when we release? It be awesome if we had a way of automating this step.

Copy link
Member Author

Choose a reason for hiding this comment

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

Agreed. Automating should be next steps

@emosbaugh emosbaugh force-pushed the emosbaugh/sc-37099/pull_request_template branch from b81382b to 1cce199 Compare September 17, 2021 18:49
@emosbaugh emosbaugh marked this pull request as ready for review September 17, 2021 18:49
@emosbaugh emosbaugh requested review from a team and murphybytes September 17, 2021 18:49
#### Does this PR require documentation?
<!--
If no, just write "NONE" below.
If yes, link to the related https://github.com/replicatedhq/kots.io documentation PR:
Copy link
Contributor

Choose a reason for hiding this comment

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

I think it's reasonable to expect a kots.io pr from employees, but considering everyone that submits a PR may not be, how about we ask them to attach a markdown file containing the PR documentation to the PR request?

Copy link
Member Author

Choose a reason for hiding this comment

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

Let's just see how it goes...

@emosbaugh emosbaugh merged commit 4fe89bc into master Sep 17, 2021
@emosbaugh emosbaugh deleted the emosbaugh/sc-37099/pull_request_template branch September 17, 2021 19:39
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

2 participants