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

Merged
merged 1 commit into from
Mar 5, 2020

Conversation

TomSweeneyRedHat
Copy link
Member

Add a pull request template. Modeled after CRI-O (Thanks @saschagrunert!) and
Dockers.

Signed-off-by: TomSweeneyRedHat tsweeney@redhat.com

Add a pull request template.  Modeled after CRI-O (Thanks @saschagrunert!) and
Dockers.

Signed-off-by: TomSweeneyRedHat <tsweeney@redhat.com>

Please make sure you've read and understood our contributing guidelines
(https://github.com/containers/buildah/blob/master/CONTRIBUTING.md) as well as ensuring
that all your commits are signed with `git commit -s`.
Copy link
Member

Choose a reason for hiding this comment

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

Is there anything to make this automatic?

Copy link
Member Author

Choose a reason for hiding this comment

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

I'm not sure, need to dig further. It certainly would be nice if we could pull that off.

Copy link
Member

Choose a reason for hiding this comment

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

Hm no I don't think we could automate this without having a write access token to the repository.

Copy link
Member

@saschagrunert saschagrunert left a comment

Choose a reason for hiding this comment

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

LGTM :)

@rhatdan
Copy link
Member

rhatdan commented Mar 4, 2020

bors r+

bors bot added a commit that referenced this pull request Mar 4, 2020
2201: Add Pull Request Template r=rhatdan a=TomSweeneyRedHat

Add a pull request template.  Modeled after CRI-O (Thanks @saschagrunert!) and
Dockers.

Signed-off-by: TomSweeneyRedHat <tsweeney@redhat.com>

Co-authored-by: TomSweeneyRedHat <tsweeney@redhat.com>
@bors
Copy link
Contributor

bors bot commented Mar 4, 2020

Build failed

  • cirrus-ci/success

@rhatdan
Copy link
Member

rhatdan commented Mar 5, 2020

bors retry

@bors
Copy link
Contributor

bors bot commented Mar 5, 2020

Build succeeded

@bors bors bot merged commit be998a6 into containers:master Mar 5, 2020
@TomSweeneyRedHat TomSweeneyRedHat deleted the dev/tsweeney/prreq1 branch April 4, 2020 17:33
@github-actions github-actions bot locked as resolved and limited conversation to collaborators Oct 4, 2023
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants