-
Notifications
You must be signed in to change notification settings - Fork 99
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
Release 1.3.4 #180
Release 1.3.4 #180
Conversation
Codecov ReportPatch has no changes to coverable lines. 📢 Thoughts on this report? Let us know!. |
2169372
to
89816ea
Compare
Before actually bloom-releasing this we should update the drivers to use the timeout instead of the keepalive count as this raises an error in startup. Thinking about this: Shouldn't we make the deprecation notice a warning? |
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.
LGTM and I also think we should make the deprecation notice a warning as mentioned in the other PR.
89816ea
to
ce05627
Compare
Now including #182 |
Waiting for #183. Sorry I didn't realize the warning was in two places... |
Release PR for next release.