[Gradle] Include `:tests` in `settings.gradle` by MGaetan89 · Pull Request #4906 · material-components/material-components-android
This commit makes the :tests module available in a Gradle build. This fixes false-positives in the version catalog (dexmaker not being used for example).
This commit makes the `:tests` module available in a Gradle build. This fixes false-positives in the version catalog (`dexmaker` not being used for example).
| } | ||
|
|
||
| android { | ||
| namespace "com.google.android.material" |
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
I've used the package name defined here:
| package="com.google.android.material"> |
Hey @MGaetan89
By false-positive, you mean dependencies like dexmaker being flagged as unused?
Hey @MGaetan89
By false-positive, you mean dependencies like
dexmakerbeing flagged as unused?
Yes, exactly
Okay gotcha. I see that that would be a nice fix.
I was making sure everything still builds as expected and was getting some errors running ./gradlew assembleDebug - maybe related to non-transitive R classes? Are you seeing anything similar?
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters. Learn more about bidirectional Unicode characters