1.7.1 does not depend on Kotlin 1.3.71

[reubenfirmin]

Looks like #2507 intended to set dependency to 1.3.71 but that doesn't work.

Also, please exclude dependencies of org.jetbrains.kotlinx:kotlinx-html-jvm, since it has older stdlib dependencies.

detekt - The detekt dependencies to be used for this project.
\--- io.gitlab.arturbosch.detekt:detekt-cli:1.7.1
     +--- org.jetbrains.kotlin:kotlin-stdlib:1.3.70
     |    +--- org.jetbrains.kotlin:kotlin-stdlib-common:1.3.70
     |    \--- org.jetbrains:annotations:13.0
     +--- io.gitlab.arturbosch.detekt:detekt-core:1.7.1
     |    +--- io.gitlab.arturbosch.detekt:detekt-api:1.7.1
     |    |    +--- org.jetbrains.kotlin:kotlin-compiler-embeddable:1.3.70
     |    |    |    +--- org.jetbrains.kotlin:kotlin-stdlib:1.3.70 (*)
     |    |    |    +--- org.jetbrains.kotlin:kotlin-script-runtime:1.3.70
     |    |    |    +--- org.jetbrains.kotlin:kotlin-reflect:1.3.70
     |    |    |    |    \--- org.jetbrains.kotlin:kotlin-stdlib:1.3.70 (*)
     |    |    |    +--- org.jetbrains.kotlin:kotlin-daemon-embeddable:1.3.70
     |    |    |    \--- org.jetbrains.intellij.deps:trove4j:1.0.20181211
     |    |    +--- org.jetbrains.kotlin:kotlin-stdlib:1.3.70 (*)
     |    |    \--- org.yaml:snakeyaml:1.26
     |    \--- org.jetbrains.kotlin:kotlin-stdlib:1.3.70 (*)
     +--- com.beust:jcommander:1.78
     +--- org.jetbrains.kotlinx:kotlinx-html-jvm:0.7.1
     |    +--- org.jetbrains.kotlinx:kotlinx-html-common:0.7.1
     |    +--- org.jetbrains.kotlin:kotlin-stdlib-jdk8:1.3.61
     |    |    +--- org.jetbrains.kotlin:kotlin-stdlib:1.3.61 -> 1.3.70 (*)
     |    |    \--- org.jetbrains.kotlin:kotlin-stdlib-jdk7:1.3.61
     |    |         \--- org.jetbrains.kotlin:kotlin-stdlib:1.3.61 -> 1.3.70 (*)
     |    \--- org.jetbrains.kotlin:kotlin-stdlib-common:1.3.61 -> 1.3.70
     \--- io.gitlab.arturbosch.detekt:detekt-rules:1.7.1
          +--- org.jetbrains.kotlin:kotlin-stdlib:1.3.70 (*)
          \--- io.gitlab.arturbosch.detekt:detekt-api:1.7.1 (*)

[BraisGabin]

That's not what I get from master

detekt - The detekt dependencies to be used for this project.
+--- project :detekt-cli (*)
+--- org.jetbrains.kotlin:kotlin-stdlib:1.3.71
|    +--- org.jetbrains.kotlin:kotlin-stdlib-common:1.3.71
|    \--- org.jetbrains:annotations:13.0
+--- project :detekt-core
|    +--- project :detekt-api
|    |    +--- org.jetbrains.kotlin:kotlin-compiler-embeddable:1.3.71
|    |    |    +--- org.jetbrains.kotlin:kotlin-stdlib:1.3.71 (*)
|    |    |    +--- org.jetbrains.kotlin:kotlin-script-runtime:1.3.71
|    |    |    +--- org.jetbrains.kotlin:kotlin-reflect:1.3.71
|    |    |    |    \--- org.jetbrains.kotlin:kotlin-stdlib:1.3.71 (*)
|    |    |    +--- org.jetbrains.kotlin:kotlin-daemon-embeddable:1.3.71
|    |    |    \--- org.jetbrains.intellij.deps:trove4j:1.0.20181211
|    |    +--- org.jetbrains.kotlin:kotlin-stdlib:1.3.71 (*)
|    |    \--- org.yaml:snakeyaml:1.26
|    \--- org.jetbrains.kotlin:kotlin-stdlib:1.3.71 (*)
+--- com.beust:jcommander:1.78
+--- org.jetbrains.kotlinx:kotlinx-html-jvm:0.7.1
|    +--- org.jetbrains.kotlinx:kotlinx-html-common:0.7.1
|    +--- org.jetbrains.kotlin:kotlin-stdlib-jdk8:1.3.61
|    |    +--- org.jetbrains.kotlin:kotlin-stdlib:1.3.61 -> 1.3.71 (*)
|    |    \--- org.jetbrains.kotlin:kotlin-stdlib-jdk7:1.3.61
|    |         \--- org.jetbrains.kotlin:kotlin-stdlib:1.3.61 -> 1.3.71 (*)
|    \--- org.jetbrains.kotlin:kotlin-stdlib-common:1.3.61 -> 1.3.71
\--- project :detekt-rules
     +--- org.jetbrains.kotlin:kotlin-stdlib:1.3.71 (*)
     \--- project :detekt-api (*)

[BraisGabin]

Anyway... it's strange that the kotlin's gradle bom is not updating kotlin-stdlib-jdk7 and kotlin-stdlib-jdk8... It should not be a problem, but it's strange.

[imanushin]

Probably this is because both kotlin-stdlib-jdk7 and kotlin-stdlib-jdk8 were received from kotlinx-html-jvm. And detekt references only kotlin-stdlib, e.g. another library (which is also mandatory), however without any suffixes.

[schalkms]

Is this related to #2501 ?

[BraisGabin]

I really don't think so. I think that it's not related. But we could check.

[arturbosch]

Thanks for insights, I've changed the dependencies a bit to get:

[sschuberth]

Is this related to #2501 ?

I really don't think so. I think that it's not related. But we could check.

I tried locally, and this seems to indeed also resolve #2501, but I'll wait for CI to verify.

Edit: Yes, #2501 is fixed in detekt 1.7.2 🎉

[arturbosch]

Thanks for the verification @sschuberth !