-
Notifications
You must be signed in to change notification settings - Fork 52
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 v2 support #225
base: master
Are you sure you want to change the base?
Conversation
Looks like setup tools for 2.9 has started to fail unrelated to your changes.
|
We can fix that, but I'll note that with the release of CKAN 2.11, 2.9 is no longer officially supported: https://ckan.org/blog/the-new-ckan-211-release-is-now-available |
Looks like 2.9 isn't compatible with the newer SQLAlchemy. We should probably make version-specific requirements files. |
BREAKING CHANGE: SQLAlchemy v2 is not supported in CKAN 2.9.x. CKAN 2.9.x support dropped
I've cut a release today for 1.2.0 which is compatible for 2.9 ~ 2.11. I'm ok with dropping 2.9 support via this PR, I've updated the doc's and cicd pipeline accordingly. I'd like to pull in the last pieces of work from the qld-gov-au fork prior to this being included for one last 1.x.x release. As this is major breaking change, it should bump to 2.0.0 on this being merged into master/released. |
Compatibility with the new SQLAlchemy version tested against this PR.