Skip to content

Conversation

@potiuk
Copy link
Member

@potiuk potiuk commented Apr 28, 2025

After updating providers to 2.10+ we can remove all conditions for 2.10+.


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

Copy link
Contributor

@eladkal eladkal left a comment

Choose a reason for hiding this comment

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

Nice!
I just started to work on it too :)

Missing handling of test_when_provider_min_airflow_version_is_2_10_or_higher_remove_obsolete_code in azure provider

@potiuk potiuk force-pushed the remove_v_10_plus_conditions branch from 7ee022d to 12a26be Compare April 28, 2025 12:40
@potiuk
Copy link
Member Author

potiuk commented Apr 28, 2025

Missing handling of test_when_provider_min_airflow_version_is_2_10_or_higher_remove_obsolete_code in azure provider

I will leave that one to @dabla :)

@potiuk potiuk force-pushed the remove_v_10_plus_conditions branch 2 times, most recently from 9a5b958 to b8e09c8 Compare April 28, 2025 14:41
@potiuk potiuk force-pushed the remove_v_10_plus_conditions branch 2 times, most recently from fb9acfd to 298661c Compare April 28, 2025 17:57
Copy link
Contributor

@jscheffl jscheffl left a comment

Choose a reason for hiding this comment

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

Nice cleanup!

After updating providers to 2.10+ we can remove all conditions for
2.10+.
@potiuk potiuk force-pushed the remove_v_10_plus_conditions branch from 298661c to de0b7df Compare April 28, 2025 19:42
@potiuk potiuk enabled auto-merge (squash) April 28, 2025 20:15
@potiuk potiuk merged commit ee200f8 into apache:main Apr 28, 2025
97 checks passed
@potiuk potiuk deleted the remove_v_10_plus_conditions branch April 28, 2025 22:17
mvfc pushed a commit to mvfc/airflow that referenced this pull request Apr 29, 2025
After updating providers to 2.10+ we can remove all conditions for
2.10+.
mvfc pushed a commit to mvfc/airflow that referenced this pull request Apr 29, 2025
After updating providers to 2.10+ we can remove all conditions for
2.10+.
jroachgolf84 pushed a commit to jroachgolf84/airflow that referenced this pull request Apr 30, 2025
After updating providers to 2.10+ we can remove all conditions for
2.10+.
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.

4 participants