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-6191: FIx bug when can not open project in workspace with custom … #6196

Merged
merged 3 commits into from
Sep 13, 2017

Conversation

vinokurig
Copy link
Contributor

…WORKDIR property

What does this PR do?

FIx bug when can not open project in workspace with custom WORKDIR property

What issues does this PR fix or reference?

fixes #6191

Changelog

FIx bug when can not open project in workspace with custom WORKDIR property

Release Notes

N/A

Docs PR

N/A

@vinokurig vinokurig added the kind/bug Outline of a bug - must adhere to the bug report template. label Sep 8, 2017
@@ -58,9 +61,11 @@
public GitChangesDetector(
RequestTransmitter transmitter,
FileWatcherManager manager,
ProjectManager projectManager,
Copy link
Contributor

@vparfonov vparfonov Sep 8, 2017

Choose a reason for hiding this comment

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

Use here Provider<ProjectManager>

Copy link
Contributor Author

Choose a reason for hiding this comment

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

done

@@ -55,9 +58,11 @@
public GitIndexChangedDetector(
RequestTransmitter transmitter,
FileWatcherManager manager,
ProjectManager projectManager,
Copy link
Contributor

Choose a reason for hiding this comment

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

Use here Provider<ProjectManager>

Copy link
Contributor Author

Choose a reason for hiding this comment

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

done


@Inject
public GitStatusProvider(GitConnectionFactory gitConnectionFactory) {
public GitStatusProvider(
GitConnectionFactory gitConnectionFactory, ProjectManager projectManager) {
Copy link
Contributor

Choose a reason for hiding this comment

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

Use here Provider<ProjectManager>

Copy link
Contributor Author

Choose a reason for hiding this comment

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

done

@codenvy-ci
Copy link

Build # 3609 - FAILED

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

@vinokurig
Copy link
Contributor Author

QA in progress

@codenvy-ci
Copy link

@vinokurig vinokurig merged commit 4a07971 into master Sep 13, 2017
@vinokurig vinokurig deleted the CHE-6191 branch September 13, 2017 06:38
@vinokurig vinokurig added this to the 5.18.0 milestone Sep 13, 2017
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.

Project explorer cannot open any project if WORKDIR is not set to /projects/
3 participants