Skip to content
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

[Cobertura] Parse all digits of a branch counter #12

Merged
merged 1 commit into from
Apr 13, 2023

Conversation

uhafner
Copy link
Member

@uhafner uhafner commented Apr 13, 2023

Erroneously, only the last digit has been counted in previous versions.

Fixes jenkinsci/code-coverage-api-plugin#625.

Erroneously, only the last digit has been counted in previous versions.
Fixes jenkinsci/code-coverage-api-plugin#625.
@uhafner uhafner added the bug Bugs or performance problems label Apr 13, 2023
@codecov
Copy link

codecov bot commented Apr 13, 2023

Codecov Report

Merging #12 (ec858a8) into main (6150a05) will not change coverage.
The diff coverage is 100.00%.

@@            Coverage Diff            @@
##               main      #12   +/-   ##
=========================================
  Coverage     85.55%   85.55%           
  Complexity      482      482           
=========================================
  Files            23       23           
  Lines          1343     1343           
  Branches        190      190           
=========================================
  Hits           1149     1149           
  Misses          148      148           
  Partials         46       46           
Impacted Files Coverage Δ
...edu/hm/hafner/coverage/parser/CoberturaParser.java 89.07% <100.00%> (ø)

📣 We’re building smart automated test selection to slash your CI/CD build times. Learn more

@uhafner uhafner changed the title Parse all digits of a counter before and after the slash [Cobertura] Parse all digits of a branch counter Apr 13, 2023
@uhafner uhafner merged commit 1241682 into main Apr 13, 2023
@uhafner uhafner deleted the wrong-cobertura-counters branch April 13, 2023 15:10
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
bug Bugs or performance problems
Projects
None yet
Development

Successfully merging this pull request may close these issues.

Different branch coverage between dotnet test and code coverage
1 participant