Could you add a note to our CHANGES file regarding this? Once that's there, I'm 👍

Codecov Report

Merging #1270 into master will not change coverage.
The diff coverage is n/a.

@@          Coverage Diff           @@
##           master   #1270   +/-   ##
======================================
  Coverage     100%    100%           
======================================
  Files          21      21           
  Lines        1985    1985           
======================================
  Hits         1985    1985

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 63e4245...582db1f. Read the comment docs.

@sigmavirus24 unless this is a breaking change, it's now documented in CHANGES.
Is there anything I can do to address the codecoverage drop?

fascinating.
i added a CHANGES entry, and now everything is failing 🤣

@igalic, I'm not sure what you mean about the drop in coverage, I see:

Merging #1270 into master will not change coverage.

I think codecov is marked as failed because AppVeyor failed. I'll see if I can restart that build.

And I can't. I don't know who can. 😞

Does AppVeyor react to a close->open like Travis does?

i did a push --force perhaps that confused AppVeyor & codecov?

looking better now!

🎉 Glad the close-open trick worked. :)

n.b.: This optimization might be obsoleted thru python/cpython#3684