Skip to content

fix: remove kover #7325

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

Open
wants to merge 1 commit into
base: main
Choose a base branch
from
Open

fix: remove kover #7325

wants to merge 1 commit into from

Conversation

exigow
Copy link
Collaborator

@exigow exigow commented Aug 22, 2025

Motivation:

  • Random issue with missing kover-agent.args (not sure what's the exact cause, but invalidating the cache + removing build/ helps)
  • We don’t get any real benefit from this at the moment (checking coverage is great, but we have too few tests for it to make sense to track or to require a certain % in new PRs; we can bring it back in the future)
  • The CI step was commented out anyway

Summary by cubic

Remove Kover coverage tooling from the IntelliJ extension and CI to eliminate flaky kover-agent.args errors and simplify builds. No functional changes; we aren’t tracking coverage yet and can re-enable later.

  • Refactors
    • Removed org.jetbrains.kotlinx.kover plugin and koverReport config from extensions/intellij/build.gradle.kts.
    • Removed the commented Codecov upload step from .github/workflows/jetbrains-release.yaml.

@exigow exigow requested a review from a team as a code owner August 22, 2025 10:05
@dosubot dosubot bot added the size:XS This PR changes 0-9 lines, ignoring generated files. label Aug 22, 2025
@exigow exigow force-pushed the exigow/remove-kover branch from 1d1a3a0 to 6865ba9 Compare August 22, 2025 11:46
@dosubot dosubot bot added size:S This PR changes 10-29 lines, ignoring generated files. and removed size:XS This PR changes 0-9 lines, ignoring generated files. labels Aug 22, 2025
Motivation:
* Random issue with missing `kover-agent.args` (not sure what's the exact cause, but invalidating the cache + removing `build/` helps)
* We don’t get any real benefit from this at the moment (checking coverage is great, but we have too few tests for it to make sense to track or to require a certain % in new PRs; we can bring it back in the future)
* The CI step was commented out anyway
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
size:S This PR changes 10-29 lines, ignoring generated files.
Projects
Status: Todo
Development

Successfully merging this pull request may close these issues.

1 participant