Codecov Report

Merging #2193 into master will decrease coverage by 0.01%.
The diff coverage is 87.50%.

@@            Coverage Diff             @@
##           master    #2193      +/-   ##
==========================================
- Coverage   96.50%   96.48%   -0.02%     
==========================================
  Files          16       16              
  Lines        5545     5551       +6     
==========================================
+ Hits         5351     5356       +5     
- Misses        194      195       +1     

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 abb82c5...f33c7f0. Read the comment docs.

It's curious that codecov thinks the mpl < 3.3 branch is not covered, since it should be tested by the "pinned" build.

codecov on the "pinned" tests instance did not complete for some reason:

Error: No coverage report found

so the "pinned" tests coverage report was not uploaded.

Well, I see that this phenomena is shared across many PRs: the first "pinned" job codecov command errors out and doesn't complete.

Edit: When DOCTESTS environment variable (defined in .travis.yml) is False coverage reports are not generated. #2104 changed the pinned tests to not run doctests and thus coverage is off.

Ah! Good sleuthing.