Skip to content

fix: Update README to add await to create_async_connector() call #1306

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

Closed
wants to merge 1 commit into from

Conversation

BurnzZ
Copy link

@BurnzZ BurnzZ commented Jun 21, 2025

A very small fix to the Asyncpg Connection Pool when I ran the code example.

@BurnzZ BurnzZ requested a review from a team as a code owner June 21, 2025 12:45
@kgala2
Copy link
Collaborator

kgala2 commented Jul 10, 2025

Thanks for catching this @BurnzZ, I will review this change shortly!

@kgala2 kgala2 changed the title fix README: add await to create_async_connector() call chore: add await to create_async_connector() call in README Jul 11, 2025
@kgala2 kgala2 changed the title chore: add await to create_async_connector() call in README fix README: add await to create_async_connector() call Jul 11, 2025
@hessjcg hessjcg changed the title fix README: add await to create_async_connector() call fix: Update README to add await to create_async_connector() call Jul 11, 2025
@BurnzZ
Copy link
Author

BurnzZ commented Jul 14, 2025

I just realized this is a duplicate of #1301.

Hey there @lopuhin! 👋 😄

@lopuhin
Copy link

lopuhin commented Jul 17, 2025

Hey @BurnzZ glad to see you here and thanks for drawing attention to it 😄 We've been using the adapter with the databases library though a little custom adapter and it's working well for us

@kgala2
Copy link
Collaborator

kgala2 commented Jul 17, 2025

I just realized this is a duplicate of #1301.

Hey there @lopuhin! 👋 😄

Hey @BurnzZ, thanks for pointing out the duplicate PR! I appreciate you bringing that to our attention. After reviewing both, I've decided to merge @lopuhin's PR since it was submitted earlier. Thanks again for your help

@BurnzZ
Copy link
Author

BurnzZ commented Jul 19, 2025

After reviewing both, I've decided to merge @lopuhin's PR since it was submitted earlier. Thanks again for your help

Cheers for looking into it @kgala2! Closing this in favor of #1301.

We've been using the adapter with the databases library though a little custom adapter and it's working well for us

@lopuhin that's awesome! I've been using this on my side project 😄

@BurnzZ BurnzZ closed this Jul 19, 2025
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