[GitHub] [airflow] zhongjiajie edited a comment on issue #6881: [AIRFLOW-6326] Sort cli commands and arg

2020-01-04 Thread GitBox
zhongjiajie edited a comment on issue #6881: [AIRFLOW-6326] Sort cli commands and arg URL: https://github.com/apache/airflow/pull/6881#issuecomment-570850316 > What do you think about not ignoring leading dash (key= lambda d: d.lstrip('-'))? I think maybe we should keep the dash,

[GitHub] [airflow] zhongjiajie edited a comment on issue #6881: [AIRFLOW-6326] Sort cli commands and arg

2020-01-04 Thread GitBox
zhongjiajie edited a comment on issue #6881: [AIRFLOW-6326] Sort cli commands and arg URL: https://github.com/apache/airflow/pull/6881#issuecomment-570850316 > What do you think about not ignoring leading dash (key= lambda d: d.lstrip('-'))? I think maybe we should keep the dash, when

[GitHub] [airflow] zhongjiajie edited a comment on issue #6881: [AIRFLOW-6326] Sort cli commands and arg

2020-01-03 Thread GitBox
zhongjiajie edited a comment on issue #6881: [AIRFLOW-6326] Sort cli commands and arg URL: https://github.com/apache/airflow/pull/6881#issuecomment-570749730 I run `test/test_core.py` local in Airflow breeze, and found out unittest `test_local_task_job` failed random , the failed reason

[GitHub] [airflow] zhongjiajie edited a comment on issue #6881: [AIRFLOW-6326] Sort cli commands and arg

2019-12-23 Thread GitBox
zhongjiajie edited a comment on issue #6881: [AIRFLOW-6326] Sort cli commands and arg URL: https://github.com/apache/airflow/pull/6881#issuecomment-568624951 Mistake commit with others patch I submit, force-push to start Travis-CI