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

Standardize AwsLambda #25100

Merged
merged 3 commits into from
Jul 19, 2022
Merged

Standardize AwsLambda #25100

merged 3 commits into from
Jul 19, 2022

Conversation

eladkal
Copy link
Contributor

@eladkal eladkal commented Jul 15, 2022

The aws_lambda.py is not the standard file path. The aws_ prefix is not needed.
For the file name I used the same path as the hook.


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

The `aws_lambda.py` is not the standard file path. The `aws_` prefix is not needed.
For the file name I used the same path as the hook.
@eladkal eladkal requested a review from potiuk July 15, 2022 19:22
@eladkal eladkal requested review from ashb and mik-laj as code owners July 15, 2022 19:43
@ferruzzi
Copy link
Contributor

Thanks for this.

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.

5 participants