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 2020/11/30 08:00:10 UTC

[GitHub] [airflow] baxievski commented on pull request #12096: add xcom push for ECSOperator

baxievski commented on pull request #12096:
URL: https://github.com/apache/airflow/pull/12096#issuecomment-735620876


   > Could you please add tests. 
   
   Sure, I'll need to understand how the operator is currently tested.
   
   > There are many branches returning `None`. Wouldn't it be possible and more clearer if we combine them?
   
   Yes, please let me know if you think it's ok now.
   
   
   
   
   


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

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