You are viewing a plain text version of this content. The canonical link for it is here.
Posted to commits@airflow.apache.org by GitBox <gi...@apache.org> on 2020/02/28 10:04:23 UTC
[GitHub] [airflow] zhongjiajie commented on a change in pull request #7148:
[AIRFLOW-6472] Correct short and long option in cli
zhongjiajie commented on a change in pull request #7148: [AIRFLOW-6472] Correct short and long option in cli
URL: https://github.com/apache/airflow/pull/7148#discussion_r385607297
##########
File path: UPDATING.md
##########
@@ -59,6 +59,26 @@ https://developers.google.com/style/inclusive-documentation
-->
+### Cli use exactly single character for short option
+
+Use exactly one single character for short option in Airflow cli, New commands are available according to the following table:
+
+| Old command | New command |
+|----------------------------------------------------|---------------------------------------------------|
+| ``airflow (dags|tasks|scheduler) [-sd, --subdir]`` | ``airflow (dags|tasks|scheduler) [-S, --subdir]`` |
+| ``airflow tasks test [-dr, --dry_run]`` | ``airflow tasks test [-dr, --dry_run]`` |
+| ``airflow dags backfill [-dr, --dry_run]`` | ``airflow dags backfill [-n, --dry_run]`` |
+| ``airflow tasks clear [-dx, --dag_regex]`` | ``airflow tasks clear [-R, --dag_regex]`` |
+| ``airflow kerberos [-kt, --keytab]`` | ``airflow kerberos [-k, --keytab]`` |
+| ``airflow tasks run [-int, --interactive]`` | ``airflow tasks run [-N, --interactive]`` |
Review comment:
Put in the same header? I thought we should make the laster change in the TOP before. So the correct way is combine the similar change?
----------------------------------------------------------------
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