-
Notifications
You must be signed in to change notification settings - Fork 261
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
SQLAlchemy 2.0 #530
Comments
@madkinsz Apologies as missed this one. So as far as I could understand, its not planned to give support to SQLA 2.0? |
We need someone to maintain this package. 🤷 |
I'm not able to review a PR with that bump here... Any of you are? @rafalp @aminalaee |
If PR's comes up that bumps SQLAlchemy support to 2.0, I'm fine with it. But I have no resources to develop one myself. |
@rafalp @Kludex no worries. Give me a few days and I will have a look and open a PR with a possible integration with SQLA 2.0 We are looking at possible backwards incompatible changes. Last time I tried a few things were changed internally within SQLA. I don't see a reason to have forks of this package just to add SQLA 2.0 when databases is already maintained |
Cool. 👍 Thanks. 🙏 |
No problem. I have products depending of databases and having the support for 2.0 we can leverage python 3. 2.0 does not support python 2 anymore. |
any update on this issue? I'd really like to use slack-sdk which depends on |
Thanks @zanieb ! I ended up finding a workaround for the time being |
`python3Packages.databases` is broken due to encode/databases#530. Luckily, it turns out that `databases` was not a necessary dependency anyways. See slackapi/python-slack-sdk#1388.
@samuela Could you share some details of your workaround? 🙏🏻 |
Can you guys tell us if you plan to add support for sqlalchemy 2.0 ? |
@ImEins there is no maintainer that is going to implement it as this time. I will review a contribution adding support. |
update? |
The work at #539 is stalled as the author is busy with other things. |
We did this. #540. Is there a timeframe for a release? @tomchristie |
@ansipunk there is one PR opened yesterday that requires some attention. Hopefully today or tomorrow can be merged and we can perform a release. |
Will there be support for SQLAlchemy 2.0 or I should just use it directly?
The text was updated successfully, but these errors were encountered: