Skip to content

Conversation

@Prab-27
Copy link
Contributor

@Prab-27 Prab-27 commented Aug 19, 2025

When this failed it missed the filepath

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.

This PR adds the missing filepath

Related discussion: here


^ Add meaningful description above
Read the Pull Request Guidelines for more information.
In case of fundamental code changes, an Airflow Improvement Proposal (AIP) is needed.
In case of a new dependency, check compliance with the ASF 3rd Party License Policy.
In case of backwards incompatible changes please leave a note in a newsfragment file, named {pr_number}.significant.rst or {issue_number}.significant.rst, in airflow-core/newsfragments.

@boring-cyborg boring-cyborg bot added area:dev-tools backport-to-v3-1-test Mark PR with this label to backport to v3-1-test branch labels Aug 19, 2025
@potiuk potiuk merged commit 55abbeb into apache:main Aug 19, 2025
48 checks passed
github-actions bot pushed a commit that referenced this pull request Aug 19, 2025
(cherry picked from commit 55abbeb)

Co-authored-by: Pratiksha <128999446+Prab-27@users.noreply.github.com>
@github-actions
Copy link

Backport successfully created: v3-0-test

Status Branch Result
v3-0-test PR Link

potiuk pushed a commit that referenced this pull request Aug 19, 2025
(cherry picked from commit 55abbeb)

Co-authored-by: Pratiksha <128999446+Prab-27@users.noreply.github.com>
mangal-vairalkar pushed a commit to mangal-vairalkar/airflow that referenced this pull request Aug 30, 2025
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

area:dev-tools backport-to-v3-1-test Mark PR with this label to backport to v3-1-test branch

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants