Skip to content

Fix failing MinSQLAlchemy test after moving sqlalchemy spec - #38445

Merged
amoghrajesh merged 1 commit into
mainfrom
fix-failing-sqlalchemy-test
Mar 25, 2024
Merged

Fix failing MinSQLAlchemy test after moving sqlalchemy spec#38445
amoghrajesh merged 1 commit into
mainfrom
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.

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.
@potiukpotiuk added canary When set on PR running from apache repo - behave as canary run default versions only When assigned to PR - only default python version is used for CI tests labels Mar 25, 2024
@potiuk
potiuk requested review from ashb and kaxil as code ownersMarch 25, 2024 01:19

@amoghrajeshamoghrajesh left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Nice catch!

@amoghrajesh
amoghrajesh merged commit a9cfa84 into mainMar 25, 2024
@Taragolis
Taragolis deleted the fix-failing-sqlalchemy-test branch March 25, 2024 07:55
Sign up for freeto join this conversation on GitHub. Already have an account? Sign in to comment

Labels

area:dev-toolscanaryWhen set on PR running from apache repo - behave as canary rundefault versions onlyWhen assigned to PR - only default python version is used for CI tests

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants

@potiuk@amoghrajesh@dirrao