You are viewing a plain text version of this content. The canonical link for it is here.
Posted to commits@airflow.apache.org by GitBox <gi...@apache.org> on 2021/11/23 21:30:33 UTC

[GitHub] [airflow] uranusjr opened a new pull request #19791: Revert "Adjust built-in base_aws methods to avoid Deprecation warnings (#19725)"

uranusjr opened a new pull request #19791:
URL: https://github.com/apache/airflow/pull/19791


   This reverts commit 4be04143a5f7e246127e942bf1d73abcd22ce189. The PR was merged prematurely.


-- 
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.

To unsubscribe, e-mail: commits-unsubscribe@airflow.apache.org

For queries about this service, please contact Infrastructure at:
users@infra.apache.org



[GitHub] [airflow] jedcunningham closed pull request #19791: Revert "Adjust built-in base_aws methods to avoid Deprecation warnings (#19725)"

Posted by GitBox <gi...@apache.org>.
jedcunningham closed pull request #19791:
URL: https://github.com/apache/airflow/pull/19791


   


-- 
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.

To unsubscribe, e-mail: commits-unsubscribe@airflow.apache.org

For queries about this service, please contact Infrastructure at:
users@infra.apache.org



[GitHub] [airflow] eladkal edited a comment on pull request #19791: Revert "Adjust built-in base_aws methods to avoid Deprecation warnings (#19725)"

Posted by GitBox <gi...@apache.org>.
eladkal edited a comment on pull request #19791:
URL: https://github.com/apache/airflow/pull/19791#issuecomment-977559978


   @dimon222 we had to revert the PR as it caused main branch to fail. Could you raise a new PR?


-- 
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.

To unsubscribe, e-mail: commits-unsubscribe@airflow.apache.org

For queries about this service, please contact Infrastructure at:
users@infra.apache.org



[GitHub] [airflow] eladkal commented on pull request #19791: Revert "Adjust built-in base_aws methods to avoid Deprecation warnings (#19725)"

Posted by GitBox <gi...@apache.org>.
eladkal commented on pull request #19791:
URL: https://github.com/apache/airflow/pull/19791#issuecomment-977559978


   @dimon222 we had to revert the PR. Could you raise a new PR and lets take a deeper look this time? :) 


-- 
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.

To unsubscribe, e-mail: commits-unsubscribe@airflow.apache.org

For queries about this service, please contact Infrastructure at:
users@infra.apache.org



[GitHub] [airflow] eladkal merged pull request #19791: Revert "Adjust built-in base_aws methods to avoid Deprecation warnings (#19725)"

Posted by GitBox <gi...@apache.org>.
eladkal merged pull request #19791:
URL: https://github.com/apache/airflow/pull/19791


   


-- 
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.

To unsubscribe, e-mail: commits-unsubscribe@airflow.apache.org

For queries about this service, please contact Infrastructure at:
users@infra.apache.org



[GitHub] [airflow] github-actions[bot] commented on pull request #19791: Revert "Adjust built-in base_aws methods to avoid Deprecation warnings (#19725)"

Posted by GitBox <gi...@apache.org>.
github-actions[bot] commented on pull request #19791:
URL: https://github.com/apache/airflow/pull/19791#issuecomment-977186447


   The PR is likely OK to be merged with just subset of tests for default Python and Database versions without running the full matrix of tests, because it does not modify the core of Airflow. If the committers decide that the full tests matrix is needed, they will add the label 'full tests needed'. Then you should rebase to the latest main or amend the last commit of the PR, and push it with --force-with-lease.


-- 
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.

To unsubscribe, e-mail: commits-unsubscribe@airflow.apache.org

For queries about this service, please contact Infrastructure at:
users@infra.apache.org