Skip to content
New issue

Have a question about this project? # for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “#”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? # to your account

packaging: SQL taps now require SQLAlchemy 2.0+ #2797

Merged

Conversation

edgarrmondragon
Copy link
Collaborator

@edgarrmondragon edgarrmondragon commented Dec 4, 2024

@edgarrmondragon edgarrmondragon added this to the v0.43 milestone Dec 4, 2024
Copy link

codecov bot commented Dec 4, 2024

Codecov Report

All modified and coverable lines are covered by tests ✅

Project coverage is 91.27%. Comparing base (d3dcc8e) to head (8372a88).
Report is 3 commits behind head on main.

Additional details and impacted files
@@           Coverage Diff           @@
##             main    #2797   +/-   ##
=======================================
  Coverage   91.27%   91.27%           
=======================================
  Files          62       62           
  Lines        5182     5182           
  Branches      669      669           
=======================================
  Hits         4730     4730           
  Misses        319      319           
  Partials      133      133           

☔ View full report in Codecov by Sentry.
📢 Have feedback on the report? Share it here.

@edgarrmondragon edgarrmondragon self-assigned this Dec 4, 2024
@edgarrmondragon edgarrmondragon force-pushed the edgarrmondragon/packaging/sql-taps-sqlalchemy-2 branch from e9ccf97 to 8372a88 Compare December 4, 2024 16:50
@edgarrmondragon edgarrmondragon marked this pull request as ready for review December 4, 2024 16:56
@edgarrmondragon edgarrmondragon requested review from a team as code owners December 4, 2024 16:56
@edgarrmondragon edgarrmondragon merged commit 2a6bee5 into main Dec 4, 2024
32 checks passed
@edgarrmondragon edgarrmondragon deleted the edgarrmondragon/packaging/sql-taps-sqlalchemy-2 branch December 4, 2024 16:56
# for free to join this conversation on GitHub. Already have an account? # to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

1 participant