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

CHE-5621: Avoid using ProjectManager just for taking project root dir #5636

Merged
merged 1 commit into from
Jul 13, 2017

Conversation

vparfonov
Copy link
Contributor

What does this PR do?

Avoid using ProjectManager only for taking project root dir instead of we will use special properties

What issues does this PR fix or reference?

#5621

@vparfonov vparfonov added kind/bug Outline of a bug - must adhere to the bug report template. team/ide labels Jul 13, 2017
@vparfonov vparfonov added this to the 5.15.0 milestone Jul 13, 2017
@vparfonov vparfonov requested review from svor and dkuleshov July 13, 2017 11:51
pom.xml Outdated
@@ -32,7 +32,7 @@
<module>plugins</module>
<module>ide/che-ide-core</module>
<module>samples</module>
<module>dashboard</module>
<!--module>dashboard</module-->
Copy link
Member

Choose a reason for hiding this comment

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

should dashboard be disabled?

Copy link
Contributor Author

Choose a reason for hiding this comment

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

fixed

…f we will use special properties

Signed-off-by: Vitalii Parfonov <vparfonov@redhat.com>
@vparfonov vparfonov merged commit 95a09d6 into master Jul 13, 2017
@vparfonov vparfonov deleted the che#5621 branch July 13, 2017 12:39
@codenvy-ci
Copy link

Build # 3065 - FAILED

Please check console output at https://ci.codenvycorp.com/job/che-pullrequests-build/3065/ to view the results.

JPinkney pushed a commit to JPinkney/che that referenced this pull request Aug 17, 2017
…f we will use special properties (eclipse-che#5636)

Signed-off-by: Vitalii Parfonov <vparfonov@redhat.com>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
kind/bug Outline of a bug - must adhere to the bug report template.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants