Skip to content
This repository has been archived by the owner on Apr 4, 2023. It is now read-only.

Revert "Run Che command in selected container if container doesn't specified in configuration" #268

Merged
merged 1 commit into from
Jun 10, 2019

Conversation

mmorhun
Copy link
Contributor

@mmorhun mmorhun commented Jun 10, 2019

Reverts #253 because it depends on this PR and do not work without that changes.

@monaka if you are not sure what and how a PR does, do not merge it please. If you are affected somehow, please ask about PR state.

@mmorhun mmorhun requested a review from evidolob June 10, 2019 06:57
@mmorhun mmorhun merged commit cf9b494 into master Jun 10, 2019
@mmorhun mmorhun deleted the revert-253-che-theia-222 branch June 10, 2019 07:25
@evidolob
Copy link
Contributor

@monaka Please do not merge pull request that you not own

@dneary
Copy link

dneary commented Jun 10, 2019

@mmorhun I'm not sure how @monaka was supposed to know that. There was nothing in PR #253 to say that it depended on a different PR, and 2 committers had approved it. I think most commits are merged by committers who do not own them, @evidolob - I think it is even best practice that you do not merge your own PRs.

monaka pushed a commit to PizzaFactory/che-theia that referenced this pull request Jun 14, 2019
vinokurig pushed a commit that referenced this pull request Apr 6, 2021
…o registry.access.redhat.com/rhscl/httpd-24-rhel7:2.4-105 (#268)

Change-Id: Ieeec90d0be83c93743affb8c191c238b44952920
Signed-off-by: nickboldt <[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.

3 participants