Skip to content

Fix issues related to kotlin-stdlib's Gradle module metadata in Kotlin 1.9.20#6590

Merged
3flex merged 2 commits into
detekt:mainfrom
3flex:stdlib-updates
Nov 1, 2023
Merged

Fix issues related to kotlin-stdlib's Gradle module metadata in Kotlin 1.9.20#6590
3flex merged 2 commits into
detekt:mainfrom
3flex:stdlib-updates

Conversation

@3flex

@3flex 3flex commented Nov 1, 2023

Copy link
Copy Markdown
Member

Gradle module metadata is published alongside kotlin-stdlib from 1.9.20 which caused a couple of dependency issues due to our use of useCompileClasspathVersions for consistent dependency resolution.

This fixes them and will unblock 1.9.20 update.

Test fixtures aren't published and there's no need to specify this
dependency for internal consumers.
@3flex 3flex added the pick request Marker for PRs that should be ported to the 1.0 release branch label Nov 1, 2023
@detekt-ci detekt-ci added the build label Nov 1, 2023
@codecov

codecov Bot commented Nov 1, 2023

Copy link
Copy Markdown

Codecov Report

All modified and coverable lines are covered by tests ✅

Comparison is base (8fd9e93) 85.18% compared to head (efa95e6) 85.18%.

Additional details and impacted files
@@            Coverage Diff            @@
##               main    #6590   +/-   ##
=========================================
  Coverage     85.18%   85.18%           
  Complexity     4061     4061           
=========================================
  Files           565      565           
  Lines         13321    13321           
  Branches       2401     2401           
=========================================
  Hits          11347    11347           
  Misses          773      773           
  Partials       1201     1201           

☔ View full report in Codecov by Sentry.
📢 Have feedback on the report? Share it here.

kotlin-stdlib is now published with Gradle module metadata which has
caused a new issue because we use consistent dependency resolution (see
useCompileClasspathVersions). Updating the version of the dependency in the
compile classpath fixes it.

KT-49803 is already open to request removing annotations as a transitive
dependency. Until that's fixed this setup will allow builds to operate
correctly.
Comment thread detekt-gradle-plugin/build.gradle.kts
@3flex 3flex merged commit 20c1fcb into detekt:main Nov 1, 2023
@3flex 3flex deleted the stdlib-updates branch November 1, 2023 10:21
@detekt-ci

Copy link
Copy Markdown
Collaborator
Warnings
⚠️ This PR is approved with no milestone set. If merged, it won't appear in the detekt release notes.

Generated by 🚫 dangerJS against efa95e6

@TWiStErRob TWiStErRob added this to the 2.0.0 milestone Nov 1, 2023
cortinico pushed a commit that referenced this pull request Nov 25, 2023
…n 1.9.20 (#6590)

* Remove redundant dependency

Test fixtures aren't published and there's no need to specify this
dependency for internal consumers.

* Use latest annotations version in compile classpath

kotlin-stdlib is now published with Gradle module metadata which has
caused a new issue because we use consistent dependency resolution (see
useCompileClasspathVersions). Updating the version of the dependency in the
compile classpath fixes it.

KT-49803 is already open to request removing annotations as a transitive
dependency. Until that's fixed this setup will allow builds to operate
correctly.
mgroth0 pushed a commit to mgroth0/detekt that referenced this pull request Feb 11, 2024
…n 1.9.20 (detekt#6590)

* Remove redundant dependency

Test fixtures aren't published and there's no need to specify this
dependency for internal consumers.

* Use latest annotations version in compile classpath

kotlin-stdlib is now published with Gradle module metadata which has
caused a new issue because we use consistent dependency resolution (see
useCompileClasspathVersions). Updating the version of the dependency in the
compile classpath fixes it.

KT-49803 is already open to request removing annotations as a transitive
dependency. Until that's fixed this setup will allow builds to operate
correctly.
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

build pick request Marker for PRs that should be ported to the 1.0 release branch

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants