potiuk commented on pull request #22613: URL: https://github.com/apache/airflow/pull/22613#issuecomment-1082985913
I think rich is flexible enough to give us full compatibility. There are few watchouts though: 1) performance - some of our commands require to import a lot of stuff and they are lazy initialized 2) auito-complete - there are some quirks with auto-complete for rich and rich-click which we have not solved yet entirely in Breeze 2 https://github.com/apache/airflow/issues/21164 - basically seems that you need to activate virtualenv with rich and rich-click for the autocomplete to work, which is pretty unfortunate (but maybe we can find some workarounds and maybe this is not an issue for Airflow as we can assume venv of airflow with rtch and rich-click will be activated before (but it might be a problem when airflow is installed via `pipx` which is more and more likely as people learn the benefits of `pipx` for installing standalone tools. -- 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]
