We read every piece of feedback, and take your input very seriously.
To see all available qualifiers, see our documentation.
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
kaml
kaml uses kotlinx.serialization and it leads to dependency issue in ktlint running is due to sarif4k.
kotlinx.serialization
So, the main problem is in ktlint:
ktlint
| \--- io.github.detekt.sarif4k:sarif4k:0.0.1 | +--- org.jetbrains.kotlinx:kotlinx-serialization-json:1.1.0 | | \--- org.jetbrains.kotlinx:kotlinx-serialization-json-jvm:1.1.0 | | +--- org.jetbrains.kotlin:kotlin-stdlib:1.4.30 -> 1.7.20 (*) | | +--- org.jetbrains.kotlin:kotlin-stdlib-common:1.4.30 -> 1.7.20 | | \--- org.jetbrains.kotlinx:kotlinx-serialization-core:1.1.0 | | \--- org.jetbrains.kotlinx:kotlinx-serialization-core-jvm:1.1.0 | | +--- org.jetbrains.kotlin:kotlin-stdlib:1.4.30 -> 1.7.20 (*) | | \--- org.jetbrains.kotlin:kotlin-stdlib-common:1.4.30 -> 1.7.20
It uses https://github.com/detekt/sarif4k which uses the old kotlinx serialization. There is a PR to upgrade it: detekt/sarif4k#4
https://github.com/detekt/sarif4k
After it, need to upgrade it in ktlint
Originally posted by @nulls in #1518 (comment)
The text was updated successfully, but these errors were encountered:
@nulls Is this issue still relevant? Seems like kotlinx.serialization was updated in detekt/safif4k a long time ago
detekt/safif4k
Sorry, something went wrong.
I don't know -- need to check dependencies
closed by shading kotlinx.serialization in #1654
nulls
Successfully merging a pull request may close this issue.
kaml
useskotlinx.serialization
and it leads to dependency issue in ktlint running is due to sarif4k.So, the main problem is in
ktlint
:It uses
https://github.com/detekt/sarif4k
which uses the old kotlinx serialization.There is a PR to upgrade it: detekt/sarif4k#4
After it, need to upgrade it in
ktlint
Originally posted by @nulls in #1518 (comment)
The text was updated successfully, but these errors were encountered: