phi-friday commented on PR #38942: URL: https://github.com/apache/airflow/pull/38942#issuecomment-2051236708
> from https://pypi.org/project/sqlalchemy2-stubs/ > > > The stubs replace the use of the “sqlalchemy-stubs” package published by Dropbox. Differences include that these stubs are generated against 1.4’s API as well as some adjustments to the use of generics. **The sqlalchemy2-stubs stubs package will not work with SQLAlchemy 2.0, and will need to be explicitly / manually uninstalled when upgrading to SQLAlchemy 2.0.** in [document](https://docs.sqlalchemy.org/en/20/orm/extensions/mypy.html) > For these reasons, new non-regression issues reported against the Mypy plugin are unlikely to be fixed. Existing code that passes Mypy checks using the plugin with SQLAlchemy 1.4 installed will continue to pass all checks in SQLAlchemy 2.0 without any changes required, provided the plugin is still used. SQLAlchemy 2.0’s API is fully backwards compatible with the SQLAlchemy 1.4 API and Mypy plugin behavior. If im only using 2.x, it makes sense to explicitly remove it, but if im using both, I thought this was best for now. However, the two documents seem to contradict each other. Regardless, it seems easier to apply it first and then remove it than to change everything at once. It would be great if i could just use `Mapped`. By the way, I've thought of a way to remove `Hint`, so I'll apply that and push again. -- 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]
