Skip to content

Conversation

@bastien-phi
Copy link
Contributor

@bastien-phi bastien-phi commented Oct 1, 2020

Some small mistakes can happen... This PR fixes one of them.

It can conflict with #817 in tests so I will have to merge master into it if this PR is accepted

@codecov
Copy link

codecov bot commented Oct 1, 2020

Codecov Report

Merging #818 into master will not change coverage.
The diff coverage is 100.00%.

Impacted file tree graph

@@            Coverage Diff            @@
##             master     #818   +/-   ##
=========================================
  Coverage     83.08%   83.08%           
  Complexity     1094     1094           
=========================================
  Files            62       62           
  Lines          3559     3559           
=========================================
  Hits           2957     2957           
  Misses          602      602           
Impacted Files Coverage Δ Complexity Δ
src/Report/Cobertura.php 95.61% <100.00%> (ø) 20.00 <0.00> (ø)

Continue to review full report at Codecov.

Legend - Click here to learn more
Δ = absolute <relative> (impact), ø = not affected, ? = missing data
Powered by Codecov. Last update ca932dd...8723a21. Read the comment docs.

@sebastianbergmann sebastianbergmann merged commit 4e93a9a into sebastianbergmann:master Oct 1, 2020
@bastien-phi bastien-phi deleted the fix-line-and-branch-rate-in-cobertura branch October 1, 2020 08:30
bastien-phi added a commit to bastien-phi/php-code-coverage that referenced this pull request Oct 1, 2020
@sebastianbergmann
Copy link
Owner

It can conflict with #817 in tests so I will have to merge master into it if this PR is accepted

Can you please send a new pull request that fixes the failing test(s)? Thanks!

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.

2 participants