kacpermuda commented on code in PR #69671: URL: https://github.com/apache/airflow/pull/69671#discussion_r3558012762
########## airflow-e2e-tests/tests/airflow_e2e_tests/openlineage_tests/prepare_dags.py: ########## @@ -30,41 +30,21 @@ from __future__ import annotations -import os import shutil from pathlib import Path -HERE = Path(__file__).resolve().parent -AIRFLOW_ROOT_PATH = HERE.parents[1] +from airflow_e2e_tests.constants import AIRFLOW_ROOT_PATH -# Example DAGs that need a minimum Airflow version. When running against an older version, the DAG Review Comment: Let's see when the compat are back if this chunk and other version-related from this file are needed, may need to be re-added. ########## providers-e2e-tests/openlineage/README.md: ########## Review Comment: Can we keep this README close to the OL tests? Even thought the layout needs adjusting, it still explain how the tests are executed and the logic behind it, very useful for debugging 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. To unsubscribe, e-mail: [email protected] For queries about this service, please contact Infrastructure at: [email protected]
