Re: [DISCUSS] Airflow cli option style

2020-02-28 Thread Kaxil Naik
Thanks Jiajie, that looks great On Fri, Feb 28, 2020 at 2:59 AM Jiajie Zhong wrote: > Hey guys, I add some code on it and finally pass Travis-CI, maybe you > could task a look at https://github.com/apache/airflow/pull/7148 < > https://github.com/apache/airflow/pull/7148> > Best Wish > — Jiajie >

Re: [DISCUSS] Airflow cli option style

2020-02-27 Thread Jiajie Zhong
Hey guys, I add some code on it and finally pass Travis-CI, maybe you could task a look at https://github.com/apache/airflow/pull/7148 Best Wish — Jiajie

Re: [DISCUSS] Airflow cli option style

2020-01-22 Thread Kaxil Naik
Let the discussion remain open a little longer. I haven't yet read much details but is in my "todo" list. On Thu, Jan 16, 2020 at 4:43 PM Jarek Potiuk wrote: > I really like the consistency - and especially that you added tests that > maintain the consistency forward :) > I am all for the change

Re: [DISCUSS] Airflow cli option style

2020-01-16 Thread Jarek Potiuk
I really like the consistency - and especially that you added tests that maintain the consistency forward :) I am all for the change - just a few comments in the PR. J. On Thu, Jan 16, 2020 at 3:40 AM hotmail wrote: > Hi everyone, I create a PR for our cli option in > https://github.com/apache/

[DISCUSS] Airflow cli option style

2020-01-15 Thread hotmail
Hi everyone, I create a PR for our cli option in https://github.com/apache/airflow/pull/7148 The initial purpose is change our cli short option to exactly single character, and PR already did it as follow | Old command