[GitHub] [airflow] turbaszek commented on pull request #9464: Fix DockerOperator xcom

2020-10-05 Thread GitBox
turbaszek commented on pull request #9464: URL: https://github.com/apache/airflow/pull/9464#issuecomment-703861860 I would be in favor of 2. + deprecation warning. I think we should return strings. @feluelle @mik-laj any opinions?

[GitHub] [airflow] turbaszek commented on pull request #9464: Fix DockerOperator xcom

2020-10-04 Thread GitBox
turbaszek commented on pull request #9464: URL: https://github.com/apache/airflow/pull/9464#issuecomment-703246220 > @nullhack I think in all cases TestDockerOperator.test_execute_xcom_behavior is failing Yes: ```

[GitHub] [airflow] turbaszek commented on pull request #9464: Fix DockerOperator xcom

2020-10-01 Thread GitBox
turbaszek commented on pull request #9464: URL: https://github.com/apache/airflow/pull/9464#issuecomment-702360591 Hey @nullhack the test are failing, can you take a look? This is an automated message from the Apache Git

[GitHub] [airflow] turbaszek commented on pull request #9464: Fix DockerOperator xcom

2020-09-13 Thread GitBox
turbaszek commented on pull request #9464: URL: https://github.com/apache/airflow/pull/9464#issuecomment-691715460 Hey @nullhack can you rebase, please? In the meantime we introduced black formatter in providers packages :)