Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Remove astunparse dependency #43560

Merged
merged 1 commit into from
Oct 31, 2024
Merged

Conversation

kaxil
Copy link
Member

@kaxil kaxil commented Oct 31, 2024

Since the minimium required Python in Airflow main is now >=3.9 since #42766, we can drop this dep


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

Since the minimium required Python in Airflow main is now >=3.9 since apache#42766, we can drop this dep
@kaxil kaxil requested review from potiuk and ashb as code owners October 31, 2024 21:10
@kaxil kaxil requested a review from rawwar October 31, 2024 21:11
@potiuk
Copy link
Member

potiuk commented Oct 31, 2024

Also nice! :)

@kaxil kaxil merged commit 3d36dc6 into apache:main Oct 31, 2024
109 checks passed
@kaxil kaxil deleted the remove-astunparse branch October 31, 2024 22:53
ellisms pushed a commit to ellisms/airflow that referenced this pull request Nov 13, 2024
Sign up for free to 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.

2 participants