Skip to content
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.

Sign up for GitHub

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

Jump to bottom

Add async support? #47

Closed
ThibTrip opened this issue Dec 17, 2021 · 1 comment · Fixed by #55
Closed

Add async support? #47

ThibTrip opened this issue Dec 17, 2021 · 1 comment · Fixed by #55
Assignees
@ThibTrip

Comments

@ThibTrip
Copy link
Owner

This is a placeholder issue until I make a new pull request. I had a very convincing POC but there were a lot of changes to the library since then. Also I realized we should solve issue #44 first (which in turns means solving issue #46 first).
@ThibTrip ThibTrip self-assigned this Dec 17, 2021
@ThibTrip ThibTrip mentioned this issue Dec 17, 2021
Merged
@ThibTrip ThibTrip mentioned this issue Jan 13, 2022
Merged
8 tasks
@ThibTrip
Copy link
Owner Author

In the new PR #55 this looks already very functional and it is fully tested (well for asyncpg at least, still have to look if there are other drivers we can test).
It still needs a lot of work though.

@ThibTrip ThibTrip mentioned this issue Feb 24, 2022
Closed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees

@ThibTrip ThibTrip

Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging a pull request may close this issue.

[FEAT] Add pangres.aupsert (async variant) ThibTrip/pangres
1 participant
@ThibTrip