Skip to content

Conversation

@vincbeck
Copy link
Contributor

@vincbeck vincbeck commented Aug 12, 2024

Min Airflow version in providers moved to 2.8.0 in #41396. Some backward compatibility code in the Amazon provider package is only if the Airflow version is 2.7.1. Therefore, we can do some cleanup.


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

@boring-cyborg boring-cyborg bot added area:providers provider:amazon AWS/Amazon - related issues labels Aug 12, 2024
@vincbeck vincbeck marked this pull request as ready for review August 12, 2024 15:40
@vincbeck vincbeck force-pushed the vincbeck/aws_cleanup branch from 8056c3d to 15a13cb Compare August 12, 2024 16:00
Copy link
Contributor

@ferruzzi ferruzzi left a comment

Choose a reason for hiding this comment

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

LGTM. Tons of small changes, but I love seeing old code dropped. The unit in particular look so much better now!

Copy link
Contributor

@o-nikolas o-nikolas left a comment

Choose a reason for hiding this comment

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

Tests still aren't passing but in spirit the changes look good once things are passsing

@vincbeck vincbeck merged commit c8fc0ad into apache:main Aug 15, 2024
@vincbeck vincbeck deleted the vincbeck/aws_cleanup branch August 15, 2024 15:20
Artuz37 pushed a commit to Artuz37/airflow that referenced this pull request Aug 19, 2024
romsharon98 pushed a commit to romsharon98/airflow that referenced this pull request Aug 20, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

area:providers provider:amazon AWS/Amazon - related issues

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants