Skip to content

Conversation

@rob93c
Copy link
Member

@rob93c rob93c commented Jan 16, 2026

Removed code coverage generation and upload to Qodana, as long as its baseline.

Summary by CodeRabbit

Release Notes

  • Chores
    • Removed code coverage reporting from the build process to streamline testing workflows.
    • Simplified continuous integration pipeline by eliminating intermediate artifact transfers between analysis steps.
    • Updated Docker configuration to align with current analysis tooling requirements.

✏️ Tip: You can customize this high-level summary in your review settings.

@rob93c rob93c requested a review from MartelliEnrico January 16, 2026 21:56
@rob93c rob93c self-assigned this Jan 16, 2026
@rob93c rob93c added docker This marks issues revolving around Docker cicd The change affects CI/CD flows labels Jan 16, 2026
@coderabbitai
Copy link
Contributor

coderabbitai bot commented Jan 16, 2026

📝 Walkthrough

Walkthrough

The pull request removes Jacoco code coverage integration from the build configuration, eliminates coverage artifact transfer steps from the CI/CD workflow, removes the Qodana baseline configuration reference, and updates Docker ignore rules to reflect configuration file changes.

Changes

Cohort / File(s) Summary
Qodana and CI/CD Configuration
.dockerignore, .github/workflows/unit-test.yml
Removed qodana.baseline.json from Docker ignore and added qodana.yaml; removed coverage report upload/download steps and baseline argument (--baseline,qodana.baseline.json) from Qodana action invocation in unit test workflow.
Build Configuration
build.gradle
Removed Jacoco plugin declaration, deleted jacocoTestReport task, and removed finalizedBy(jacocoTestReport) from test task configuration.

Estimated code review effort

🎯 2 (Simple) | ⏱️ ~12 minutes

Possibly related PRs

  • Enable Qodana scans #456: Directly opposes these changes by adding Qodana baseline argument and coverage-related configuration; potential merge conflict in workflow and Qodana setup.

Suggested reviewers

  • MartelliEnrico

Poem

🐰 With hop and cheer, we simplify today,
Jacoco metrics and baselines fade away,
No coverage files to clutter the flow,
Our pipelines lighter, they'll swiftly go,
Docker and workflows, now clean and neat!

🚥 Pre-merge checks | ✅ 3
✅ Passed checks (3 passed)
Check name Status Explanation
Description Check ✅ Passed Check skipped - CodeRabbit’s high-level summary is enabled.
Title check ✅ Passed The title directly reflects the main objective of the PR: removing Jacoco code coverage generation, Qodana baseline configuration, and related CI/CD steps that are unsupported by Qodana Community Edition.
Docstring Coverage ✅ Passed No functions found in the changed files to evaluate docstring coverage. Skipping docstring coverage check.

✏️ Tip: You can configure your own custom pre-merge checks in the settings.


Thanks for using CodeRabbit! It's free for OSS, and your support helps us grow. If you like it, consider giving us a shout-out.

❤️ Share

Comment @coderabbitai help to get the list of available commands and usage tips.

Copy link
Contributor

@MartelliEnrico MartelliEnrico left a comment

Choose a reason for hiding this comment

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

LGTM

@rob93c rob93c merged commit a8bbf82 into main Jan 17, 2026
10 of 12 checks passed
@rob93c rob93c deleted the remove-coverage-generation branch January 17, 2026 09:56
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

cicd The change affects CI/CD flows docker This marks issues revolving around Docker

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants