Skip to content

Re-export RunError and PooledConnection from bb8 #35

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

Merged
merged 1 commit into from
Nov 3, 2022

Conversation

johnchildren
Copy link
Contributor

@johnchildren johnchildren commented Oct 26, 2022

Re-export the RunError type from bb8. Required when handling errors from the pool.


Edit: Updated to include the specific PoolError in the type alias and to include PooledConnection.

@johnchildren johnchildren force-pushed the re-export-bb8-run-error branch from 7ec2ca1 to 523ca2f Compare October 26, 2022 10:33
@johnchildren johnchildren force-pushed the re-export-bb8-run-error branch from 523ca2f to 5d5195d Compare November 2, 2022 16:28
@johnchildren johnchildren changed the title Re-export RunError from bb8 Re-export RunError and PooledConnection from bb8 Nov 2, 2022
@weiznich
Copy link
Owner

weiznich commented Nov 3, 2022

Thanks for opening this PR. I potentially plan to relicense diesel-async in the future. Because of that I need an explicitly written consent that you transfer the ownership of this change to me. Please add that to the commit message.

Re-export type aliases for RunError and PooledConnection for the bb8
connection pool.

I consent that I transfer the ownership of this change to weiznich
(Georg Semmler).
@johnchildren johnchildren force-pushed the re-export-bb8-run-error branch from 5d5195d to e86b86b Compare November 3, 2022 16:29
@johnchildren
Copy link
Contributor Author

Thanks for opening this PR. I potentially plan to relicense diesel-async in the future. Because of that I need an explicitly written consent that you transfer the ownership of this change to me. Please add that to the commit message.

Thank you, consent added!

@weiznich weiznich merged commit 439a5f1 into weiznich:main Nov 3, 2022
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.

2 participants