Skip to content
New issue

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

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

Already on GitHub? # to your account

add type annotations to Amazon provider "execute_coplete" methods #36330

Merged
merged 3 commits into from
Feb 16, 2024

Conversation

Lee-W
Copy link
Member

@Lee-W Lee-W commented Dec 20, 2023


^ 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 Dec 20, 2023
@Lee-W Lee-W force-pushed the amazon-provider-improvement branch from b1c5950 to 4d533b7 Compare December 20, 2023 14:38
Copy link
Contributor

@dirrao dirrao left a comment

Choose a reason for hiding this comment

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

@Lee-W
Thanks for the MR. It is looks good to me. I have given you a minor comment.

Copy link
Contributor

@dirrao dirrao left a comment

Choose a reason for hiding this comment

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

As per the @syedahsn recommendation, update the job details.

@Lee-W
Copy link
Member Author

Lee-W commented Jan 18, 2024

I'll probably defer this one a bit. I'm thinking of extracting the event check method to the base hook so that we don't need to rewrite it everywhere.

@Lee-W Lee-W force-pushed the amazon-provider-improvement branch from 4d533b7 to 0df2772 Compare February 15, 2024 10:37
@Lee-W Lee-W marked this pull request as ready for review February 15, 2024 10:40
@Lee-W Lee-W changed the title minor Amazon provider improvements add type annotations to Amazon provider "execute_coplete" methods Feb 15, 2024
@Lee-W Lee-W force-pushed the amazon-provider-improvement branch from 4c17b76 to 3198d09 Compare February 16, 2024 04:05
@Lee-W Lee-W force-pushed the amazon-provider-improvement branch from 3198d09 to 538b30c Compare February 16, 2024 06:13
Copy link
Member

@potiuk potiuk left a comment

Choose a reason for hiding this comment

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

Nice refactor.

@potiuk potiuk merged commit c75a105 into apache:main Feb 16, 2024
58 checks passed
@Lee-W Lee-W deleted the amazon-provider-improvement branch June 5, 2024 06:19
# for free to join this conversation on GitHub. Already have an account? # 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