Conversation
There was a problem hiding this comment.
Pull request overview
This pull request upgrades Kover (Kotlin code coverage library) from version 0.6.1 to 0.9.7 and migrates the configuration to the new API format introduced in the 0.9.x series. The upgrade requires changing the plugin module name, plugin ID, and completely restructuring the coverage configuration from the test task level to a new top-level reports DSL.
Changes:
- Updated Kover dependency from 0.6.1 to 0.9.7 and changed the module artifact to kover-gradle-plugin
- Migrated Kover configuration from the deprecated 0.6.x API to the new 0.9.x reports DSL
- Removed obsolete configuration options (intellij version reference, coverage engine selection, generateReportOnCheck flag)
Reviewed changes
Copilot reviewed 2 out of 2 changed files in this pull request and generated 1 comment.
| File | Description |
|---|---|
| gradle/libs.versions.toml | Updated Kover version to 0.9.7, changed module name to kover-gradle-plugin, and removed the intellij version reference that is no longer needed |
| dbinspector/build.gradle | Updated plugin ID to org.jetbrains.kotlinx.kover, migrated coverage includes/excludes from test task configuration to the new reports.filters DSL, and removed deprecated configuration options |
💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.
There was a problem hiding this comment.
Pull request overview
Copilot reviewed 2 out of 2 changed files in this pull request and generated 2 comments.
💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.
| classes( | ||
| "com.infinum.dbinspector.data.sources.*", | ||
| "com.infinum.dbinspector.domain.*.*UseCase*", | ||
| "com.infinum.dbinspector.domain.*.*Repository*", | ||
| "com.infinum.dbinspector.domain.*.*Control*", | ||
| "com.infinum.dbinspector.domain.*.*Converter*", | ||
| "com.infinum.dbinspector.domain.*.*Mapper*", | ||
| "com.infinum.dbinspector.domain.*.*Interactor*", | ||
| "com.infinum.dbinspector.ui.*.*ViewModel*" | ||
| ) |
There was a problem hiding this comment.
The class filter patterns have been modified by removing the trailing .* from patterns like "com.infinum.dbinspector.domain.*.*UseCase*.*" to "com.infinum.dbinspector.domain.*.*UseCase*". In Kover 0.9.x, the filter patterns should match fully qualified class names, not nested classes or members. The removal of the trailing .* may change the coverage scope. Please verify that these patterns still match the intended classes correctly. For example, if you have nested classes or inner classes within UseCases, Repositories, etc., the new patterns may not include them.
KCeh
force-pushed
the
housekeeping/bump-kover
branch
from
a097d49 to
b048d43
Compare
|
Summary
Simple change, bumping kover and migrating config to new version
Related issue:
Changes
Type
Additional information
Description
Checklist
Additional notes