Skip to content

ci: add Codecov coverage upload and XML report generation - #579

Merged
kpavlov merged 1 commit into
mainfrom
kpavlov/add-codecov
Mar 2, 2026
Merged

ci: add Codecov coverage upload and XML report generation#579
kpavlov merged 1 commit into
mainfrom
kpavlov/add-codecov

Conversation

@kpavlov

@kpavlov kpavlov commented Mar 2, 2026

Copy link
Copy Markdown
Contributor

Add Codecov coverage upload and XML report generation

  • Add koverXmlReport to Gradle tasks in Ubuntu workflow
  • Configure Codecov action to upload coverage reports

Reports will be available on https://app.codecov.io/gh/modelcontextprotocol/kotlin-sdk

Motivation and Context

To provide better test coverage visibility

How Has This Been Tested?

CI

Breaking Changes

No

Types of changes

  • CI 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 added the ci label Mar 2, 2026
@codecov-commenter

Copy link
Copy Markdown

Welcome to Codecov 🎉

Once you merge this PR into your default branch, you're all set! Codecov will compare coverage reports and display results in all future pull requests.

ℹ️ You can also turn on project coverage checks and project coverage reporting on Pull Request comment

Thanks for integrating Codecov - We've got you covered ☂️

@kpavlov
kpavlov force-pushed the kpavlov/add-codecov branch 2 times, most recently from c2dfe6e to 8eb19cc Compare March 2, 2026 12:31
@kpavlov
kpavlov marked this pull request as ready for review March 2, 2026 12:34
@kpavlov
kpavlov requested review from devcrocod, e5l and zarechneva-a and removed request for e5l March 2, 2026 13:17
- Add `koverXmlReport` to Gradle tasks in Ubuntu workflow
- Configure Codecov action to upload coverage and test reports

Reports will be available on https://app.codecov.io/gh/modelcontextprotocol/kotlin-sdk
@kpavlov
kpavlov force-pushed the kpavlov/add-codecov branch from 8eb19cc to d07e7df Compare March 2, 2026 13:49

@zarechneva-a zarechneva-a left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

LGTM, thank you

@kpavlov
kpavlov merged commit c368812 into main Mar 2, 2026
13 checks passed
@kpavlov
kpavlov deleted the kpavlov/add-codecov branch March 2, 2026 14:19
@devcrocod

Copy link
Copy Markdown
Contributor

Why do we need this?

We shouldn’t introduce a third-party AI tool without a clear and explicit need!

Especially considering that, instead of codecov, we could add the claude plugin or qodana, which would provide similar functionality. And likely in a more reliable way than an external AI tool. And that’s only if we actually need this in the first place

@kpavlov

kpavlov commented Mar 2, 2026

Copy link
Copy Markdown
Contributor Author

@devcrocod, just so you know, this isn’t an AI tool. It’s simply a viewer for the generated coverage XML report that Kover creates.

I don’t think there’s anything wrong with adding a reporting tool. What I’m more worried about is the lack of reporting and visibility.

Qodana is a bit slow, and I’m happy to consider switching to another tool that offers better UX.

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.

4 participants