Prab-27 commented on PR #52325: URL: https://github.com/apache/airflow/pull/52325#issuecomment-3199029228
> Just came across the CI failure. > > ``` > Prevent deprecated sqlalchemy usage.........................................Failed > - hook id: prevent-deprecated-sqlalchemy-usage > - exit code: 1 > Use of legacy `session.query` detected on line 167. > SQLAlchemy 2.0 deprecates the `Query` objectuse the `select()` construct instead. > ``` > > It would be nice to include the file path in the logging for resolving static check as followup. > > https://github.com/apache/airflow/blob/9b46d76ee4fffdbefa7fa14aba5b7d8a8ec15f3b/scripts/ci/prek/prevent_deprecated_sqlalchemy_usage.py#L29-L44 > > As current logging will only print the line number but the file path is missing. Thanks! Appreciate the explanation! I’ve had similar moments when static checks fail without clear feedback — this definitely helps -- 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]
