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

Define the 'dev' container as a target for merging a 'container contribution' #168

Closed

Conversation

RomanNikitenko
Copy link
Contributor

Signed-off-by: Roman Nikitenko rnikiten@redhat.com

What does this PR do?

Define the 'dev' container as a target for merging a 'container contribution'.
See description of the issue.

What issues does this PR fix?

eclipse-che/che#21738

How to test this PR?

After resolving eclipse-che/che#21738 we should check that a workspace starts successfully for the che-code repo.

@github-actions
Copy link

github-actions bot commented Feb 1, 2023

Click here to review and test in web IDE: Contribute

@github-actions
Copy link

github-actions bot commented Feb 1, 2023

Click here to review and test in web IDE: Contribute

…ibution'

Signed-off-by: Roman Nikitenko <rnikiten@redhat.com>
@github-actions
Copy link

github-actions bot commented Feb 6, 2023

Pull Request images published ✨
image: quay.io/che-incubator-pull-requests/che-code:pr-168-amd64
Dev image: quay.io/che-incubator-pull-requests/che-code-dev:pr-168-dev-amd64

@l0rd
Copy link
Contributor

l0rd commented Feb 10, 2023

@RomanNikitenko it's not required to specify the first container as the target for merging contributions anymore. See devfile/devworkspace-operator#993

@RomanNikitenko RomanNikitenko deleted the merge-contribution branch February 10, 2023 15:31
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.

2 participants