Codecov Report

Merging #30 into master will decrease coverage by -0.01%.
The diff coverage is n/a.

@@            Coverage Diff             @@
##           master      #30      +/-   ##
==========================================
- Coverage   82.37%   82.37%   -0.01%     
==========================================
  Files        1427     1427              
  Lines      350948   350948              
==========================================
- Hits       289088   289085       -3     
- Misses      61860    61863       +3

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 e7ffb99...15b5a34. Read the comment docs.

I'm curious what the reasoning for this is?

Several pull request threads, such as this one, have gotten noisy but irrelevant 'coverage decreased by .01 %' warnings. I am puzzled that this patch to the .yml driver should change code coverage at all. If the change shrinks the message somehow, I am in favor of it.

In my rush to get this submitted this morning, I failed to reference python/core-workflow#21, which inspired this.

@nedbat, the object here is to avoid having Codecov "fail" its check due to random noise in the coverage results. This is obviously not the ideal solution; ideally, we should find and squash the noise. This is a good way to make the coverage check more useful in the short term, though.

@terryjreedy, I don't think this will affect the messages left by codecov, but will affect the status in the 'merge' box at the bottom of the PR (note the difference between this one and #20).