Skip to content
This repository has been archived by the owner on Aug 16, 2024. It is now read-only.

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

Merged
merged 1 commit into from
Oct 6, 2020

Conversation

benoitf
Copy link
Contributor

@benoitf benoitf commented Oct 6, 2020

What does this PR do?

Add the new PR Template

What issues does this PR fix or reference?

eclipse-che/che#17486

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

Change-Id: I632d9498d6c13808a21478424e01ca6244018fbe
Signed-off-by: Florent Benoit <[email protected]>
@che-bot
Copy link
Contributor

che-bot commented Oct 6, 2020

Devfiles tests failed. Re-trigger by [ci-test-devfiles]

-->

### What does this PR do?


### Screenshot/screencast of this PR
<!-- Please include a screenshot or a screencast explaining what is doing this PR -->
Copy link
Contributor

Choose a reason for hiding this comment

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

It all looks OK to me I'm just wondering if we need a screenshot/screencast for things in this repo

Copy link
Contributor Author

Choose a reason for hiding this comment

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

it's optional but it could be what you bring with a devfile

For example if I add a quarkus devfile, then I can see a screenshot of theia + project + quarkus being started

or for a petclinic app, a screenshot of the running app

@ericwill ericwill merged commit aa3c547 into master Oct 6, 2020
@ericwill ericwill deleted the CHE-17486 branch October 6, 2020 21:29
@che-bot che-bot added this to the 7.20 milestone Oct 6, 2020
monaka pushed a commit to PizzaFactory/che-devfile-registry that referenced this pull request Oct 11, 2020
… PR (eclipse-che#289)

Change-Id: I632d9498d6c13808a21478424e01ca6244018fbe
Signed-off-by: Florent Benoit <[email protected]>
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants