Skip to content

Comments

chore: remove DatabaseWithDebugError#1545

Merged
rakita merged 1 commit intobluealloy:mainfrom
DaniPopes:rm-db-with-dbug
Jun 19, 2024
Merged

chore: remove DatabaseWithDebugError#1545
rakita merged 1 commit intobluealloy:mainfrom
DaniPopes:rm-db-with-dbug

Conversation

@DaniPopes
Copy link
Collaborator

Looks unused.

@rakita
Copy link
Member

rakita commented Jun 18, 2024

It is here to be exported, usable for someone who wants a database with debug errors and does not need to redefine this trait.

Would close this PR

@mattsse
Copy link
Collaborator

mattsse commented Jun 18, 2024

this looks unimplemented though, so not really useful

@DaniPopes
Copy link
Collaborator Author

if you're using it generically you would have either dbext like in foundry, or you can specify "Db<Error: Debug>" (now with rust 1.79 its even easier)

@rakita
Copy link
Member

rakita commented Jun 19, 2024

Convinced. Merging PR

@rakita rakita merged commit e6b52af into bluealloy:main Jun 19, 2024
This was referenced Jun 19, 2024
@DaniPopes DaniPopes deleted the rm-db-with-dbug branch June 19, 2024 13:56
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants