-
Notifications
You must be signed in to change notification settings - Fork 72
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
update to latest asyncpg
dependency
#3614
Conversation
this is obviously a straightforward change to get us past a build error. @ThomasLaPiana and @pattisdr just figured you two would be the best suited to know whether updating i've got a changelog update queued up but just gonna hold off on a push to i can see CI go green first 😅 |
Passing run #2786 ↗︎
Details:
This comment has been generated by cypress-bot as a result of this project's GitHub integration settings. |
Codecov ReportPatch and project coverage have no change.
Additional details and impacted files@@ Coverage Diff @@
## main #3614 +/- ##
=======================================
Coverage 87.06% 87.06%
=======================================
Files 310 310
Lines 19002 19002
Branches 2427 2427
=======================================
Hits 16544 16544
Misses 2029 2029
Partials 429 429 ☔ View full report in Codecov by Sentry. |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Fixed the build error for me! Thanks @adamsachs
Also verified that this fixed things, thanks for your work here! |
Thanks @adamsachs , I had this error in another project and took me a while to get it was asyncpg. |
Closes #3613
Code Changes
asyncpg
dependency to avoid build errorSteps to Confirm
Pre-Merge Checklist
CHANGELOG.md
Description Of Changes
See issue description for a synopsis of why this is just now biting us, it's been lurking for a while!