Skip to content

Conversation

@vincbeck
Copy link
Contributor

This PR is a follow up of #31135. It had been decided to split #31135 in multiple PRs to make it easier to review. This PR scope is Amazon Provider only.


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

@vincbeck
Copy link
Contributor Author

vincbeck commented Jun 2, 2023

Thanks for reviewing @uranusjr. I addressed your suggestions

@potiuk
Copy link
Member

potiuk commented Jun 4, 2023

LGTM. @uranusjr ?

@uranusjr
Copy link
Member

uranusjr commented Jun 5, 2023

Has conflicts

@vincbeck
Copy link
Contributor Author

vincbeck commented Jun 5, 2023

All green :)

@potiuk potiuk merged commit cda83c2 into apache:main Jun 5, 2023
@vincbeck vincbeck deleted the vincbeck/d400-providers-amazon branch June 5, 2023 22:39
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.

3 participants