-
Notifications
You must be signed in to change notification settings - Fork 39
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
block version of kaml-jvm #1580
Conversation
### What's done: * block version of kaml-jvm to have same version of serialization as in ktlint
Codecov Report
@@ Coverage Diff @@
## master #1580 +/- ##
=========================================
Coverage 82.43% 82.43%
Complexity 2323 2323
=========================================
Files 112 112
Lines 7845 7845
Branches 2034 2034
=========================================
Hits 6467 6467
Misses 453 453
Partials 925 925
Flags with carried forward coverage won't be shown. Click here to find out more. Help us with your feedback. Take ten seconds to tell us how you rate us. Have a feature suggestion? Share it here. |
What exactly was the problem with this? Also, does ktlint use kx.serialization at all? Or is there any transitive dependency that gets overridden? |
ktlint uses detekt sarif4k which uses kx.serialization (old version). detekt sarif4k already have a change with upgrading kx.serialization (0.2.0), but ktlint will be upgraded in version 0.48.x and more. So we will able to upgrade kaml after upgrading ktlint |
What's done: