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

Fix SQLAlchemy deprecation warnings #3049

Merged
merged 1 commit into from
Feb 13, 2025

Conversation

soininen
Copy link
Contributor

@soininen soininen commented Feb 13, 2025

Fixed two minor warnings in the execution tests.

Re spine-tools/Spine-Database-API#477

Checklist before merging

  • Documentation is up-to-date
  • Release notes have been updated
  • Unit tests have been added/updated accordingly
  • Code has been formatted by black & isort
  • Unit tests pass

Copy link

codecov bot commented Feb 13, 2025

Codecov Report

All modified and coverable lines are covered by tests ✅

Project coverage is 67.21%. Comparing base (e6d97a9) to head (3d87852).
Report is 2 commits behind head on master.

Additional details and impacted files
@@            Coverage Diff             @@
##           master    #3049      +/-   ##
==========================================
- Coverage   67.22%   67.21%   -0.02%     
==========================================
  Files         194      194              
  Lines       37284    37284              
  Branches     4475     4475              
==========================================
- Hits        25063    25059       -4     
+ Misses      10953    10952       -1     
- Partials     1268     1273       +5     

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

@soininen soininen merged commit 43a0236 into master Feb 13, 2025
21 of 22 checks passed
@soininen soininen deleted the fix_sqlalchemy_warnings_in_execution_tests branch February 13, 2025 11:41
# 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