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.

None yet

3 participants