Copilot commented on code in PR #65587:
URL: https://github.com/apache/airflow/pull/65587#discussion_r3369262276
##########
airflow-core/src/airflow/models/variable.py:
##########
@@ -46,6 +45,7 @@ class AirflowSecretsBackendAccessDenied(PermissionError): #
type: ignore[no-red
"""Compat stub — never raised by task-sdk <1.2.2."""
+from airflow.process_context import should_use_task_sdk_api_path
Review Comment:
This import is placed after executable code (the try/except + fallback class
definition above), which triggers Ruff/pycodestyle E402 (module level import
not at top of file). Please move it into the main import block at the top of
the module, or explicitly suppress E402 if the placement is intentional (e.g.
to keep the compat stub isolated).
##########
airflow-core/src/airflow/models/connection.py:
##########
@@ -49,6 +48,7 @@ class AirflowSecretsBackendAccessDenied(PermissionError): #
type: ignore[no-red
"""Compat stub — never raised by task-sdk <1.2.2."""
+from airflow.process_context import should_use_task_sdk_api_path
Review Comment:
This import is placed after executable code (the try/except + fallback class
definition above), which triggers Ruff/pycodestyle E402 (module level import
not at top of file). Please move it into the main import block at the top of
the module, or explicitly suppress E402 if the placement is intentional (e.g.
to keep the compat stub isolated).
--
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]
