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

fix finding resources in overlayed workspaces #10

Merged
merged 1 commit into from
Nov 3, 2015

Conversation

dirk-thomas
Copy link
Contributor

Before when finding any resource in a workspace aborted searching in underlayed workspaces. Instead we always have to consider all workspaces but only use the highest level entry with the same name.

@dirk-thomas dirk-thomas added the in review Waiting for review (Kanban column) label Nov 3, 2015
@dirk-thomas dirk-thomas self-assigned this Nov 3, 2015
@jacquelinekay
Copy link

If I understand the code correctly, Windows doesn't replicate the new behavior. Do we support overlayed workspaces in Windows?

@dirk-thomas
Copy link
Contributor Author

Great catch. I added the missing check in the Windows block.

@tfoote
Copy link
Member

tfoote commented Nov 3, 2015

+1

dirk-thomas added a commit that referenced this pull request Nov 3, 2015
fix finding resources in overlayed workspaces
@dirk-thomas dirk-thomas merged commit 5f40c8f into master Nov 3, 2015
@dirk-thomas dirk-thomas removed the in review Waiting for review (Kanban column) label Nov 3, 2015
@dirk-thomas dirk-thomas deleted the resources_in_overlayed_ws branch November 3, 2015 19:59
dirk-thomas added a commit that referenced this pull request Nov 3, 2015
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

3 participants