Skip to content

Conversation

@jbrockopp
Copy link
Contributor

@jbrockopp jbrockopp commented Jun 9, 2023

Dependent on #878

This change removes all the old database code which is no longer used:

This logic has been condensed into the database agnostic engine{} type introduced in #868.

@jbrockopp jbrockopp added the enhancement Indicates an improvement to a feature label Jun 9, 2023
@jbrockopp jbrockopp self-assigned this Jun 9, 2023
@codecov
Copy link

codecov bot commented Jun 9, 2023

Codecov Report

Merging #880 (24b2cf6) into main (e2e86a7) will decrease coverage by 0.52%.
The diff coverage is n/a.

Impacted file tree graph

@@            Coverage Diff             @@
##             main     #880      +/-   ##
==========================================
- Coverage   71.80%   71.28%   -0.52%     
==========================================
  Files         311      303       -8     
  Lines       13026    12374     -652     
==========================================
- Hits         9353     8821     -532     
+ Misses       3211     3127      -84     
+ Partials      462      426      -36     

@jbrockopp jbrockopp marked this pull request as ready for review June 9, 2023 14:52
@jbrockopp jbrockopp requested a review from a team as a code owner June 9, 2023 14:52
@ecrupper ecrupper merged commit 0410f86 into main Jun 9, 2023
@ecrupper ecrupper deleted the chore/database/cleanup branch June 9, 2023 21:20
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

enhancement Indicates an improvement to a feature

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants