[GitHub] [airflow] potiuk commented on issue #6881: [AIRFLOW-6326] Sort cli commands and arg

[GitBox <gi...@apache.org>]

potiuk commented on issue #6881: [AIRFLOW-6326] Sort cli commands and arg
URL: https://github.com/apache/airflow/pull/6881#issuecomment-570891939
 
 
   One other point since we are discussing cli changes and cleaning up.
   
   I just realised that we have some "short" options that are really "long" ones: namely -int and -sd in the run task. This is against the idea of short and long options in Unix (and argparse follows that). The main reason to have short options is that you can combine short options:
   
   `airflow task run -iAlm` 
   
   When you have more than one letter "short" options this might become quickly ambiguous. We do not have -s yet but if we add few more options this might become a problem.
   
   Also in the argparse documentation https://docs.python.org/2/library/argparse.html it's mentioned that short options should be single character only.
   
   Does it bother others? Or is it just me ? Should we maybe fix that?
   

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


With regards,
Apache Git Services