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

detekt-cli 1.17.0-RC2 is published with multiple runtime variants #3743

Closed
3flex opened this issue May 5, 2021 · 2 comments · Fixed by #3747
Closed

detekt-cli 1.17.0-RC2 is published with multiple runtime variants #3743

3flex opened this issue May 5, 2021 · 2 comments · Fixed by #3747

Comments

@3flex
Copy link
Member

3flex commented May 5, 2021

Expected Behavior

Single variant with "org.gradle.usage": "java-runtime" (apiElements) is published in Gradle metadata.

Observed Behavior

Two variants with "org.gradle.usage": "java-runtime" (apiElements, shadowRuntimeElements) are published in Gradle metadata.

Steps to Reproduce

Compare .module file in https://repo1.maven.org/maven2/io/gitlab/arturbosch/detekt/detekt-cli/1.16.0/ & https://repo1.maven.org/maven2/io/gitlab/arturbosch/detekt/detekt-cli/1.17.0-RC2/

Diff: https://gist.github.com/3flex/c82cf481f4d45c0bb38ee04dd769bcb8

Context

This makes it more difficult to consume detekt-cli in Gradle projects, see #3738

@3flex 3flex changed the title detekt-cli 1.17.0-RC2 is published with multiple variants detekt-cli 1.17.0-RC2 is published with multiple runtime variants May 5, 2021
@chao2zhang
Copy link
Member

chao2zhang commented May 6, 2021

It looks like the new runtime variants were introduced in #3574 - It looks like these additional variants are introduced by upgrading shadow plugin.

@3flex
Copy link
Member Author

3flex commented May 6, 2021

Thanks, that's pointed me in the right direction.

I'll have a PR soon.

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 a pull request may close this issue.

2 participants