-
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
Use kotlinx.serialization instead of jackson #393
Conversation
### What's done: * Updated kotlin * Updated ktlint * Changed kotlin-stdlib dependency to kotlin-stdlib-jdk8 * pom.xml cleanup * Fixed kotlin compiler warnings
### What's done: * Moved from jackson to kotlinx.serialization and kaml
### What's done: * detekt code style
### What's done: * Fixes
### What's done: * Fixes
### What's done: * Try to rollback to stdlib from stdlib-jdk8
### What's done: This reverts commit 75dae5b
### What's done: * Use old methods for min
### What's done: * Compile with api version 1.3
### What's done: * Use current kotlin api version
### What's done: * Use new methods instead of deprecated
### What's done: * Use apiVersion 1.3
…/kotlinx-serialization # Conflicts: # diktat-rules/src/main/kotlin/org/cqfn/diktat/ruleset/rules/identifiers/LocalVariablesRule.kt # diktat-test-framework/pom.xml
Codecov Report
@@ Coverage Diff @@
## master #393 +/- ##
============================================
- Coverage 82.55% 82.54% -0.02%
Complexity 1340 1340
============================================
Files 65 65
Lines 3251 3249 -2
Branches 1042 1043 +1
============================================
- Hits 2684 2682 -2
Misses 177 177
Partials 390 390
Flags with carried forward coverage won't be shown. Click here to find out more.
Continue to review full report at Codecov.
|
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
lgtm
No description provided.