kaxil commented on a change in pull request #14457:
URL: https://github.com/apache/airflow/pull/14457#discussion_r582899755



##########
File path: airflow/bin/cli.py
##########
@@ -414,7 +414,7 @@ def variables_export(args):
     _vars_wrapper(args, export=args.file)
 
 
-@cli_utils.deprecated_action(new_name='variables')
+@cli_utils.deprecated_action(new_name='variables list')

Review comment:
       This does not make sense to me:
   
   ```
   root@99624d92186f:/opt/airflow# airflow variables
   The 'variables' command is deprecated and removed in Airflow 2.0, please use 
'variables list' instead
   
   root@99624d92186f:/opt/airflow# airflow variables list
   usage: airflow variables [-h] COMMAND ...
   
   Manage variables
   
   positional arguments:
     COMMAND
       delete    Delete variable
       export    Export all variables
       get       Get variable
       import    Import variables
       set       Set variable
   
   optional arguments:
     -h, --help  show this help message and exit
   
   airflow variables command error: argument COMMAND: invalid choice: 'list' 
(choose from 'delete', 'export', 'get', 'import', 'set'), see help above.
   ```




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


Reply via email to