Codecov Report

All modified and coverable lines are covered by tests ✅

Project coverage is 99.94%. Comparing base (dcc6ded) to head (879eca2).
Report is 350 commits behind head on main.

@@           Coverage Diff           @@
##             main    #1066   +/-   ##
=======================================
  Coverage   99.94%   99.94%           
=======================================
  Files          34       34           
  Lines       13846    13865   +19     
=======================================
+ Hits        13839    13858   +19     
  Misses          7        7           

I'm not sure whether we should add a note somewhere in the documentation/docstrings. Other than that, this PR is ready for review.

cc:
@joshmoore

@malmans2 : I don't think I can update your branch with the latest origin/main. Could you do so to get all the tests passing?

@malmans2 : I don't think I can update your branch with the latest origin/main. Could you do so to get all the tests passing?

Done! Let me know if we should start a deprecation cycle.

Happy if someone who wants to be adventurous speaks up, but I'd err on the side of the deprecation cycle.

Sorry for the delay, @malmans2, and thanks for the deprecation cycle. If we're temporarily accepting the stringification, would you be able to workaround your TypeError by explicitly calling str()?

Thanks, @malmans2! Rolling into 2.13