Skip to content

PR and master coverage comparison #140

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.

Sign up for GitHub

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

Jump to bottom
Merged
miroslavpojer merged 16 commits into from
May 20, 2025
Merged

PR and master coverage comparison #140

miroslavpojer merged 16 commits into from
May 20, 2025

Conversation

@miroslavpojer
Copy link
Contributor

@miroslavpojer miroslavpojer commented Apr 3, 2025

Release Notes:

  • Added steps and baseline input to introduce PR and master code cove…rage comparison.

Closes #139

@miroslavpojer miroslavpojer self-assigned this Apr 3, 2025
@github-actions
Copy link

github-actions bot commented Apr 3, 2025 •
edited
Loading

JaCoCo code coverage report - Scala 2.11

Metric (instruction) Coverage Threshold Δ Coverage Status
Overall 76.8% 75.0% 0.0% ✅
Changed Files 0.0% 80.0% 0.0% ✅
Report Coverage (O/Ch) Threshold (O/Ch) Δ Coverage (O/Ch) Status (O/Ch)
ABSA Commons - scala:2.11.12 76.8% / 0.0% 75.0% / 80.0% 0.0% / 0.0% ✅/✅
File Path Coverage Threshold Δ Coverage Status

No changed file in reports.

@github-actions
Copy link

github-actions bot commented Apr 3, 2025 •
edited
Loading

JaCoCo code coverage report - Scala 2.12

Metric (instruction) Coverage Threshold Δ Coverage Status
Overall 75.23% 75.0% -0.05% ✅
Changed Files 0.0% 80.0% 0.0% ✅
Report Coverage (O/Ch) Threshold (O/Ch) Δ Coverage (O/Ch) Status (O/Ch)
ABSA Commons - scala:2.12.15 75.23% / 0.0% 75.0% / 80.0% -0.05% / 0.0% ✅/✅
File Path Coverage Threshold Δ Coverage Status

No changed file in reports.

@miroslavpojer miroslavpojer marked this pull request as ready for review April 3, 2025 18:12
@sonarqubecloud
Copy link

sonarqubecloud bot commented May 12, 2025

Quality Gate Passed Quality Gate passed

Issues
0 New issues
0 Accepted issues

Measures
0 Security Hotspots
0.0% Coverage on New Code
0.0% Duplication on New Code

See analysis details on SonarQube Cloud

dk1844
dk1844 approved these changes May 12, 2025
View reviewed changes
Copy link
Contributor

@dk1844 dk1844 left a comment

Choose a reason for hiding this comment

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

LGTM (read the code)

@miroslavpojer miroslavpojer merged commit 491324b into master May 20, 2025
17 checks passed
@miroslavpojer miroslavpojer deleted the feature/139-Introduce-usage-of-master-baseline-in-JaCoCo branch May 20, 2025 05:41
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

@dk1844 dk1844 dk1844 approved these changes

@lsulak lsulak lsulak approved these changes

@Zejnilovic Zejnilovic Awaiting requested review from Zejnilovic

@benedeki benedeki Awaiting requested review from benedeki

Assignees

@miroslavpojer miroslavpojer

Labels

None yet

Projects

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

Introduce usage of master baseline in JaCoCo

4 participants

@miroslavpojer @dk1844 @lsulak