feluelle commented on a change in pull request #7572: [AIRFLOW-6948] Remove ASCII Airflow from version command URL: https://github.com/apache/airflow/pull/7572#discussion_r385596550
########## File path: airflow/cli/commands/version_command.py ########## @@ -16,9 +16,8 @@ # under the License. """Version command""" import airflow -from airflow import settings def version(args): """Displays Airflow version at the command line""" - print(settings.HEADER + " v" + airflow.__version__) + print(f"{airflow.__version__}") Review comment: ```suggestion print(airflow.__version__) ``` Wait a second.. why are we now still using f string if it is only one variable without any extra string attached to it? 😁 ---------------------------------------------------------------- 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: us...@infra.apache.org With regards, Apache Git Services