Skip to content

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
@github-project-automation github-project-automation bot moved this from Todo to In Progress in Issues and PRs Aug 25, 2025
@dosubot dosubot bot added the lgtm This PR has been approved by a maintainer label Aug 25, 2025
@Patrick-Erichsen Patrick-Erichsen merged commit 43ea6ae into main Aug 25, 2025
39 of 41 checks passed
@Patrick-Erichsen Patrick-Erichsen deleted the exigow/remove-kover branch August 25, 2025 14:20
@github-project-automation github-project-automation bot moved this from In Progress to Done in Issues and PRs Aug 25, 2025
@github-actions github-actions bot locked and limited conversation to collaborators Aug 25, 2025
@sestinj
Copy link
Contributor

sestinj commented Aug 25, 2025

🎉 This PR is included in version 1.10.0 🎉

The release is available on:

Your semantic-release bot 📦🚀

@sestinj
Copy link
Contributor

sestinj commented Aug 27, 2025

🎉 This PR is included in version 1.11.0 🎉

The release is available on:

Your semantic-release bot 📦🚀

Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
lgtm This PR has been approved by a maintainer released size:S This PR changes 10-29 lines, ignoring generated files.
Projects
Status: Done
Development

Successfully merging this pull request may close these issues.

3 participants