jscheffl commented on code in PR #46891:
URL: https://github.com/apache/airflow/pull/46891#discussion_r2159399421
##########
airflow-core/tests/unit/serialization/test_dag_serialization.py:
##########
@@ -386,6 +386,11 @@ def get_excluded_patterns() -> Generator[str, None, None]:
if python_version in provider_info.get("excluded-python-versions"):
provider_path = provider.replace(".", "/")
yield f"providers/{provider_path}"
+ # CURRENT_PYTHON_VERSION =
f"{sys.version_info.major}.{sys.version_info.minor}"
+ # if CURRENT_PYTHON_VERSION == "3.13":
+ # We should remove google when ray is fixed to work with Python 3.13
Review Comment:
Is this a TODO post merge (issue ticket?) or to be fixed before merge?
##########
airflow-core/tests/unit/serialization/test_dag_serialization.py:
##########
@@ -411,6 +416,7 @@ def collect_dags(dag_folder=None):
excluded_patterns = [
f"{AIRFLOW_REPO_ROOT_PATH}/{excluded_pattern}" for excluded_pattern in
get_excluded_patterns()
]
+ # print("Excluded patterns:", excluded_patterns)
Review Comment:
Debug code leftovers?
--
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]
