[GitHub] [airflow] turbaszek commented on a change in pull request #12375: More proper exit status for failed CLI requests

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

turbaszek commented on a change in pull request #12375:
URL: https://github.com/apache/airflow/pull/12375#discussion_r523786298



##########
File path: airflow/cli/commands/variable_command.py
##########
@@ -65,7 +65,7 @@ def variables_import(args):
     if os.path.exists(args.file):
         _import_helper(args.file)
     else:
-        print("Missing variables file.")
+        raise SystemExit("Missing variables file.")

Review comment:
       ```suggestion
           raise SystemExit("\n\tMissing variables file.")
   ```
   If we want to be consistent... but to be honest I'm not sure if we need this "prefix".




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