Skip to content

Fix failing MinSQLAlchemy test after moving sqlalchemy spec - #38444

Closed
potiuk wants to merge 1 commit into
apache:mainfrom
potiuk:fix-failing-sqlalchemy-test
Closed

Fix failing MinSQLAlchemy test after moving sqlalchemy spec#38444
potiuk wants to merge 1 commit into
apache:mainfrom
potiuk:fix-failing-sqlalchemy-test

Conversation

@potiuk

Copy link
Copy Markdown
Member

Ths MIN SQLACLHEMY test was based on retrieving sqlalchemy min version from pyproject.toml but since we moved it to hatch_build.py we should read it from there.


^ 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 newsfragments.

Comment thread.github/workflows/run-unit-tests.yml Outdated
@potiuk
potiukforce-pushed the fix-failing-sqlalchemy-test branch 2 times, most recently from f933afc to 3a71097CompareMarch 25, 2024 01:17
Ths MIN SQLACLHEMY test was based on retrieving sqlalchemy min
version from pyproject.toml but since we moved it to hatch_build.py
we should read it from there.
@potiuk
potiukforce-pushed the fix-failing-sqlalchemy-test branch from 3a71097 to fa1ec4cCompareMarch 25, 2024 01:18
@potiukpotiuk closed this Mar 25, 2024
@potiuk

Copy link
Copy Markdown
MemberAuthor

Closed in favour of canary test in #38445

@potiuk
potiuk deleted the fix-failing-sqlalchemy-test branch April 21, 2026 15:56
Sign up for freeto join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant

@potiuk