-
Notifications
You must be signed in to change notification settings - Fork 322
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
fix(demo): make python_quickstart/demo.py work on sqlalchemy 2.0.27 #3979
fix(demo): make python_quickstart/demo.py work on sqlalchemy 2.0.27 #3979
Conversation
… to make python_quickstart/demo.py works
Codecov ReportAll modified and coverable lines are covered by tests ✅
Additional details and impacted files@@ Coverage Diff @@
## main #3979 +/- ##
=============================================
+ Coverage 41.61% 74.79% +33.18%
Complexity 711 711
=============================================
Files 196 762 +566
Lines 11790 137456 +125666
Branches 2073 2073
=============================================
+ Hits 4906 102811 +97905
- Misses 6581 34342 +27761
Partials 303 303 ☔ View full report in Codecov by Sentry. |
SDK Test Report102 files 102 suites 2m 15s ⏱️ Results for commit 878fcfb. ♻️ This comment has been updated with latest results. |
e280f13
to
e61ee63
Compare
Linux Test Report 60 files 253 suites 1h 56m 59s ⏱️ Results for commit 878fcfb. ♻️ This comment has been updated with latest results. |
e61ee63
to
32b575b
Compare
32b575b
to
878fcfb
Compare
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
Bug fix for
What is the current behavior? (You can also link to an open issue here)
The demo script does not work on SQLAclmey 2.0.27
What is the new behavior (if this is a feature change)?
Make python_quickstart/demo.py work again.