potiuk commented on PR #48673: URL: https://github.com/apache/airflow/pull/48673#issuecomment-2786928507
We can't (or rather should not) limit sqlalchemy to <3 because we did not run full tests on Sqlalchemy 2. It's deliberately kept to <2 because we have not yet run Airflow with it. After the `Fab` removal from "preinstalled" we could potentially run such tests - but they will not run in CI for now because we have the other dependencies (fab dependencies) constraining us from upgrading to SQLAlchemy 2 We can add similar special test to run the core tests on sqlalchemy 2 - similarly as we do for aiobotocore but this should be a separate PR adding it, and removing the limitation. I opened an issue for it https://github.com/apache/airflow/issues/48953 @dependabot ignore this major version -- This is an automated message from the Apache Git Service. To respond to the message, please log on to GitHub and use the URL above to go to the specific comment. To unsubscribe, e-mail: [email protected] For queries about this service, please contact Infrastructure at: [email protected]
