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

chore(github): Update the pull request template to match new definition of done on PR #17627

Merged
merged 1 commit into from
Sep 11, 2020

Conversation

benoitf
Copy link
Contributor

@benoitf benoitf commented Aug 13, 2020

What does this PR do?

Update the pull request template to match with the new definition of done on pull requests.

Screenshot/screencast of this PR

N/A

What issues does this PR fix or reference?

Fixes #17486

How to test this PR?

you can try by opening PR against the repository https://github.com/benoitf/che-template-issues

I'm also using the PR template for this PR so you can see it live :-)

PR Checklist

As the author of this Pull Request I made sure that:

Reviewers

Reviewers, please comment how you tested the PR when approving it.

Change-Id: I34dd7dead00e75dafd905b632bd24b81a9be24f1
Signed-off-by: Florent Benoit [email protected]

@che-bot che-bot added status/code-review This issue has a pull request posted for it and is awaiting code review completion by the community. kind/task Internal things, technical debt, and to-do tasks to be performed. labels Aug 13, 2020
@benoitf
Copy link
Contributor Author

benoitf commented Aug 13, 2020

[crw-ci-test]

@eclipse-che eclipse-che deleted a comment from che-bot Aug 13, 2020
@che-bot
Copy link
Contributor

che-bot commented Aug 13, 2020

✅ E2E Happy path tests succeed 🎉

See Details

Tested with Eclipse Che Multiuser User on K8S (minikube v1.1.1)

.github/PULL_REQUEST_TEMPLATE.md Show resolved Hide resolved
.github/PULL_REQUEST_TEMPLATE.md Outdated Show resolved Hide resolved
.github/PULL_REQUEST_TEMPLATE.md Outdated Show resolved Hide resolved
.github/PULL_REQUEST_TEMPLATE.md Outdated Show resolved Hide resolved
.github/PULL_REQUEST_TEMPLATE.md Outdated Show resolved Hide resolved
.github/PULL_REQUEST_TEMPLATE.md Outdated Show resolved Hide resolved
.github/PULL_REQUEST_TEMPLATE.md Outdated Show resolved Hide resolved
.github/PULL_REQUEST_TEMPLATE.md Outdated Show resolved Hide resolved
.github/PULL_REQUEST_TEMPLATE.md Outdated Show resolved Hide resolved
@che-bot
Copy link
Contributor

che-bot commented Sep 7, 2020

✅ E2E Happy path tests succeed 🎉

See Details

Tested with Eclipse Che Multiuser User on K8S (minikube v1.1.1)

@che-bot
Copy link
Contributor

che-bot commented Sep 7, 2020

✅ E2E Happy path tests succeed 🎉

See Details

Tested with Eclipse Che Multiuser User on K8S (minikube v1.1.1)

@l0rd
Copy link
Contributor

l0rd commented Sep 10, 2020

@benoitf I have submitted #17834. As a side note I have changed "Tests are covering the bugfix or new feature" with "Tests are covering the bugfix". That's because I don't want to enforce the creation of new tests for every PR but only if it fixes a bug.

…on of done on PR

eclipse-che#17486

Change-Id: I34dd7dead00e75dafd905b632bd24b81a9be24f1
Signed-off-by: Florent Benoit <[email protected]>
@benoitf
Copy link
Contributor Author

benoitf commented Sep 11, 2020

@l0rd PR has been updated

I've added links to each element to CONTRIBUTING.md file

I've also updated the description of this PR to match the content of the template to see how it looks live

@l0rd
Copy link
Contributor

l0rd commented Sep 11, 2020

@benoitf LGTM

@che-bot
Copy link
Contributor

che-bot commented Sep 11, 2020

✅ E2E Happy path tests succeed 🎉

See Details

Tested with Eclipse Che Multiuser User on K8S (minikube v1.1.1)

@benoitf benoitf merged commit 9c5d915 into eclipse-che:master Sep 11, 2020
@che-bot che-bot removed the status/code-review This issue has a pull request posted for it and is awaiting code review completion by the community. label Sep 11, 2020
@che-bot che-bot added this to the 7.19 milestone Sep 11, 2020
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
kind/task Internal things, technical debt, and to-do tasks to be performed.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

Definition of done for Pull Requests
8 participants