ad-m commented on code in PR #63561:
URL: https://github.com/apache/airflow/pull/63561#discussion_r2935431045
##########
dev/breeze/src/airflow_breeze/utils/virtualenv_utils.py:
##########
@@ -37,14 +37,14 @@ def create_venv(
capture_output=True,
)
if venv_command_result.returncode != 0:
- get_console().print(
+ console_print(
f"[error]Error when initializing virtualenv in
{venv_path.as_posix()}:[/]\n"
f"{venv_command_result.stdout}\n{venv_command_result.stderr}"
)
sys.exit(venv_command_result.returncode)
python_path = venv_path / "bin" / "python"
if not python_path.exists():
- get_console().print(f"\n[errors]Python interpreter is not exist in
path {python_path}. Exiting!\n")
+ console_print(f"\n[errors]Python interpreter is not exist in path
{python_path}. Exiting!\n")
sys.exit(1)
Review Comment:
Out of scope.
##########
dev/breeze/src/airflow_breeze/utils/path_utils.py:
##########
@@ -447,11 +447,11 @@ def cleanup_python_generated_files():
"You can also remove those files manually using sudo."
)
else:
- get_console().print("[warnings]There were files that you could not
clean-up:\n")
- get_console().print(permission_errors)
- get_console().print("You can also remove those files manually
using sudo.")
+ console_print("[warnings]There were files that you could not
clean-up:\n")
+ console_print(permission_errors)
+ console_print("You can also remove those files manually using
sudo.")
Review Comment:
Out of scope.
--
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.
To unsubscribe, e-mail: [email protected]
For queries about this service, please contact Infrastructure at:
[email protected]
