potiuk commented on code in PR #54258: URL: https://github.com/apache/airflow/pull/54258#discussion_r2277226647
########## scripts/ci/prek/compile_ui_assets_dev.py: ########## @@ -22,13 +22,13 @@ import sys from pathlib import Path -sys.path.insert(0, str(Path(__file__).parent.resolve())) # make sure common_precommit_utils is imported -from common_precommit_utils import AIRFLOW_CORE_SOURCES_PATH, AIRFLOW_ROOT_PATH +sys.path.insert(0, str(Path(__file__).parent.resolve())) # make sure common_prek_utils is imported +from common_prek_utils import AIRFLOW_CORE_SOURCES_PATH, AIRFLOW_ROOT_PATH -# NOTE!. This script is executed from node environment created by pre-commit and this environment +# NOTE!. This script is executed from a node environment created by a prek hook, and this environment # Cannot have additional Python dependencies installed. We should not import any of the libraries -# here that are not available in stdlib! You should not import common_precommit_utils.py here because -# They are importing rich library which is not available in the node environment. +# here that are not available in stdlib! You should not import common_prek_utils.py here because +# They are importing the rich library which is not available in the node environment. Review Comment: ACK. -- 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]
