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

Add root project folder to each key in projectTreeItemMap #1617

Merged
merged 1 commit into from
Oct 16, 2024

Conversation

jjohannes
Copy link
Contributor

@jjohannes jjohannes commented Oct 14, 2024

In #1612, the keys of this map were changed to the Gradle project path. This path is unique for each project within one Gradle build, but not if there are multiple Gradle builds loaded in one workspace by using: 'gradle.nestedProjects': true

This change combined the old and new behavior by creating the Map key from two parts:

  1. The root project folder which uniquely identifies the build (accessible via definition.projectFolder)
  2. The Gradle project path that uniquely identifies a project inside the build

Fixes #1614 by adjusting a change made in #1612

In microsoft#1612, the keys of this map were changed to the Gradle project path.
This path is unique for each project within one Gradle build, but not
if there are multiple Gradle builds loaded in one workspace by using:
'gradle.nestedProjects': true

This change combined the old and new behavior by creating the Map key
from two parts:
(1) The root project folder which uniquely identifies the build
    (accessible via definition.projectFolder)
(2) The Gradle project path that uniquely identifies a project inside
    the build

Signed-off-by: Jendrik Johannes <[email protected]>
@jdneo jdneo added this to the 3.16.5 milestone Oct 16, 2024
@jdneo jdneo added the bug Something isn't working label Oct 16, 2024
Copy link
Member

@jdneo jdneo left a comment

Choose a reason for hiding this comment

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

LGTM

@jdneo jdneo merged commit 9542e91 into microsoft:develop Oct 16, 2024
8 checks passed
@jdneo
Copy link
Member

jdneo commented Oct 16, 2024

Thank you @jjohannes !

@jjohannes jjohannes deleted the 1614-project-hierarchy-fix branch October 16, 2024 07:00
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
bug Something isn't working
Projects
None yet
Development

Successfully merging this pull request may close these issues.

sub projects belong to wrong folder in explorer
2 participants