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 #1342

Merged
merged 2 commits into from
May 20, 2021
Merged

Add pull request template #1342

merged 2 commits into from
May 20, 2021

Conversation

dfarrell07
Copy link
Member

Add a template for PRs with tips and pointers to docs.

This completes a recommendation of the CII Best Practices program:

It is SUGGESTED that this policy on adding tests (see test_policy) be
documented in the instructions for change proposals.

Depends on: submariner-io/submariner-website#515
Signed-off-by: Daniel Farrell dfarrell@redhat.com

Add a template for PRs with tips and pointers to docs.

This completes a recommendation of the CII Best Practices program:

> It is SUGGESTED that this policy on adding tests (see test_policy) be
documented in the instructions for change proposals.

Signed-off-by: Daniel Farrell <dfarrell@redhat.com>
@dfarrell07 dfarrell07 added this to the 0.10-m1 milestone May 19, 2021
@dfarrell07 dfarrell07 self-assigned this May 19, 2021
@submariner-bot
Copy link
Contributor

🤖 Created branch: z_pr1342/dfarrell07/pr_template

@dfarrell07 dfarrell07 marked this pull request as ready for review May 19, 2021 15:31
@dfarrell07 dfarrell07 requested a review from nyechiel May 19, 2021 15:31
@dfarrell07
Copy link
Member Author

dfarrell07 commented May 19, 2021

One issue with this, maybe something to fix/extend, is that I think the commit message (if there's only one) will not be provided as the default PR message, which is quite helpful IMHO.

@dfarrell07 dfarrell07 mentioned this pull request May 19, 2021
87 tasks
@skitt skitt merged commit fc0eea3 into submariner-io:devel May 20, 2021
@skitt skitt deleted the pr_template branch May 20, 2021 13:36
@submariner-bot
Copy link
Contributor

🤖 Closed branches: [z_pr1342/dfarrell07/pr_template]

@dfarrell07
Copy link
Member Author

Actually this works even better than I expected, the commit message is still provided as a default. 🥳

2021-05-20-095304_912x685_scrot

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

Successfully merging this pull request may close these issues.

6 participants