Upgrade unit tests - #520
Conversation
Codecov Report
@@ Coverage Diff @@
## master #520 +/- ##
=======================================
Coverage 90.84% 90.84%
=======================================
Files 28 28
Lines 2664 2664
=======================================
Hits 2420 2420
Misses 244 244
Continue to review full report at Codecov.
|
|
The "pre-commit.ci - pr" check failed because the pre-commit is checking against master instead of a tag here: https://github.com/jazzband/tablib/pull/520/files#diff-63a9c44a44acf85fea213a857769990937107cf072831e1a26808cfde9d096b9R14 We need to use Let's just remove the pre-commit check for now, and can add it back when there's a release. |
5c779b8 to
d86456a
Compare
|
I've released 0.4.2 with the changes in the pre-commit hook: https://github.com/isidentical/teyit/releases/tag/0.4.2 |
d86456a to
a73a261
Compare
|
Thanks for the new release! "pre-commit.ci - pr" now errors because it seems it's running on Python 3.8. I've opened an issue to ask how we can set a Python version at pre-commit.ci: pre-commit-ci/issues#117 |
|
pre-commit.ci now defaults to Python 3.10 and passes: |
I used https://github.com/isidentical/teyit to upgrade the unit tests to use methods which give more useful error messages upon failure.