[GitHub] [airflow] vandonr-amz opened a new pull request, #27874: better warning messages when running breeze in rosetta

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

vandonr-amz opened a new pull request, #27874:
URL: https://github.com/apache/airflow/pull/27874

   a couple of fixes:
    - if the message was not answered, an ugly traceback was printed to the console because the timeout exception was not `except`ed
    - somehow, the prompt was not displayed in a rosetta terminal (not sure why because [the code includes a flush](https://github.com/johejo/inputimeout/blob/master/inputimeout/inputimeout.py#L18), but anyway), so I added a newline character, which triggers a flush.
    - rephrased "press y" into "answer with y" because pressing the y key is not enough, one has to press enter after that, which is more obvious in the new formulation imo.


-- 
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: commits-unsubscribe@airflow.apache.org

For queries about this service, please contact Infrastructure at:
users@infra.apache.org

[GitHub] [airflow] eladkal merged pull request #27874: better warning messages when running breeze in rosetta

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

eladkal merged PR #27874:
URL: https://github.com/apache/airflow/pull/27874


-- 
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: commits-unsubscribe@airflow.apache.org

For queries about this service, please contact Infrastructure at:
users@infra.apache.org