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 Android Studio's "Duplicate content roots detected" warning. #767

Closed
wants to merge 1 commit into from

Conversation

kdeus
Copy link
Contributor

@kdeus kdeus commented Oct 31, 2024

This resources directory is included in both the testapp main and commonMain modules (by KMM). Android Studio's response to detecting the same directory in multiple modules is to remove it from one of them: the main module, in this case. This change ends up with the same result: we no longer add this directory to the main module (it's only in commonMain).

Tests still pass, so as far as I can tell this hasn't broken anything. But I haven't been able to test iOS or any other platform than Android.

Tested by:

  • ./gradlew check

  • ./gradlew connectedCheck

  • Tests pass

This resources directory is included in both the testapp main and commonMain
modules. Android Studio's response to detecting the same directory in multiple
modules is to remove it from one of them: the main module, in this case. This
change ends up with the same result: we no longer add this directory to the
main module (it's only in commonMain).

Tests still pass, so as far as I can tell this hasn't broken anything. But I
haven't been able to test iOS or any other platform than Android.

Tested by:
- ./gradlew check
- ./gradlew connectedCheck

Signed-off-by: Kevin Deus <[email protected]>
@kdeus kdeus requested a review from davidz25 October 31, 2024 21:15
@kdeus
Copy link
Contributor Author

kdeus commented Oct 31, 2024

n/m, this breaks the build.

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.

1 participant