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:
[email protected]


With regards,
Apache Git Services

Reply via email to