-
Notifications
You must be signed in to change notification settings - Fork 8
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
use renamed functions from typepigeon>=2.0.0
#62
base: main
Are you sure you want to change the base?
Conversation
>=2.0.0
>=2.0.0
typepigeon>=2.0.0
Hi @zacharyburnett thanks for working on this. In the last merge (#60) I made sure all the tests pass (except for the coverage that depends on time of year that your run). Please let me know if you'd like me to contribute to this pull, thanks! |
No problem! It looks like the test failures are due to using Once that is resolved, this should run fine. |
Codecov ReportAttention: Patch coverage is
Additional details and impacted files@@ Coverage Diff @@
## main #62 +/- ##
=======================================
Coverage 91.70% 91.70%
=======================================
Files 19 19
Lines 1988 1988
=======================================
Hits 1823 1823
Misses 165 165 ☔ View full report in Codecov by Sentry. |
@zacharyburnett I reran the failed tests after |
95ae1e8
to
e077818
Compare
e077818
to
5562180
Compare
Fixes #59