Skip to content

Conversation

kpavlov
Copy link
Contributor

@kpavlov kpavlov commented Sep 11, 2025

Motivation and Context

Make sure, that code is properly tested

How Has This Been Tested?

Run ./gradlew clean ktlintCheck build koverVerify koverLog koverHtmlReport on CI

Changes

  • Integrate Kover plugin for code coverage analysis
  • Publish Html report
  • Run ./gradlew clean ktlintCheck build koverVerify koverLog koverHtmlReport on CI

Types of changes

  • Bug fix (non-breaking change which fixes an issue)
  • New feature (non-breaking change which adds functionality)
  • Breaking change (fix or feature that would cause existing functionality to change)
  • Documentation update
  • Build tooling update

Checklist

  • I have read the MCP Documentation
  • My code follows the repository's style guidelines
  • New and existing tests pass locally
  • I have added appropriate error handling
  • I have added or updated documentation as needed

Additional context

@kpavlov kpavlov requested review from sdubov and devcrocod September 11, 2025 05:55
@kpavlov kpavlov changed the title Add Kover(age) check Add Kover check Sep 11, 2025
@kpavlov kpavlov force-pushed the kpavlov/kover branch 2 times, most recently from f9eee84 to f3faee8 Compare September 11, 2025 07:04
- Integrate Kover plugin for code coverage analysis
- Run Kover tasks on CI
- Publish Html report
- Update actions/checkout@v5
- Add `ktlintCheck`, `koverLog`, and `koverHtmlReport` to Gradle build step in CI.
- Increase minimum coverage threshold from 60% to 65%.
@kpavlov kpavlov marked this pull request as ready for review September 11, 2025 10:34
Copy link
Contributor

@devcrocod devcrocod left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

👍

@kpavlov kpavlov merged commit 6b5f7a7 into main Sep 11, 2025
4 checks passed
@kpavlov kpavlov deleted the kpavlov/kover branch September 11, 2025 10:43
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants