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

Correct resolving of current project path #588

Merged
merged 1 commit into from
Dec 19, 2019
Merged

Correct resolving of current project path #588

merged 1 commit into from
Dec 19, 2019

Conversation

vzhukovs
Copy link
Contributor

@vzhukovs vzhukovs commented Dec 17, 2019

What does this PR do?

This changes proposal modifies behavior of resolving ${current.project.path} by respecting current selection and presence of projects in project root. Besides if user didn't select any project, the warning message will be shown.

Signed-off-by: Vladyslav Zhukovskyi vzhukovs@redhat.com

What issues does this PR fix or reference?

eclipse-che/che#13636

Release Notes

N/A

Docs PR

N/A

Signed-off-by: Vladyslav Zhukovskyi <vzhukovs@redhat.com>
@che-bot
Copy link
Contributor

che-bot commented Dec 19, 2019

✅ E2E Happy path tests succeed 🎉

Try Che-Theia editor only Try Che-Theia with Java/maven example Try Che-Theia with NodeJs example

See Details

name link
che-theia docker.io/maxura/che-theia:588
che-theia-endpoint-runtime-binary docker.io/maxura/che-theia-endpoint-runtime-binary:588

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

@che-bot
Copy link
Contributor

che-bot commented Dec 19, 2019

✅ E2E Happy path tests succeed 🎉

Try Che-Theia editor only Try Che-Theia with Java/maven example Try Che-Theia with NodeJs example

See Details

name link
che-theia docker.io/maxura/che-theia:588
che-theia-endpoint-runtime-binary docker.io/maxura/che-theia-endpoint-runtime-binary:588

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

@vzhukovs
Copy link
Contributor Author

ci-build

@vzhukovs vzhukovs changed the title Macro Correct resolving of current project path Dec 19, 2019
@vzhukovs vzhukovs merged commit 38578f4 into master Dec 19, 2019
@vzhukovs vzhukovs deleted the macro branch December 19, 2019 14:11
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.

3 participants