Uh oh!
There was an error while loading. Please reload this page.
Removed deprecated code from Alibaba provider package - #44576
Conversation
Congratulations on your first Pull Request and welcome to the Apache Airflow community! If you have any issues or are unsure about any anything please check our Contributors' Guide (https://github.com/apache/airflow/blob/main/contributing-docs/README.rst)
|
shahar1
left a comment
There was a problem hiding this comment.
Welcome to Apache Airflow and thank you for your contribution, great job!
Could you please remove the now redundant imports? :)
Uh oh!
There was an error while loading. Please reload this page.
Uh oh!
There was an error while loading. Please reload this page.
potiuk
commented
Dec 10, 2024
That's the LAST one @Tushar4432 -> shall you rebase/apply the comments so that we can close #44559 🚀 ? |
1de66ad to
405e723Compareremoved depricated code from alibaba provider
Co-authored-by: Shahar Epstein <60007259+shahar1@users.noreply.github.com>
405e723 to
3f69ddeCompareAwesome work, congrats on your first merged pull request! You are invited to check our Issue Tracker for additional contributions. |
* issue 44559 removed depricated code from alibaba provider * Apply suggestions from code review Co-authored-by: Shahar Epstein <60007259+shahar1@users.noreply.github.com> --------- Co-authored-by: Elad Kalif <45845474+eladkal@users.noreply.github.com> Co-authored-by: Shahar Epstein <60007259+shahar1@users.noreply.github.com>
Relates to: #44559
Removed deprecated code from the Alibaba providers.
^ 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.