jscheffl commented on code in PR #64981:
URL: https://github.com/apache/airflow/pull/64981#discussion_r3060773841
##########
providers/common/ai/src/airflow/providers/common/ai/plugins/hitl_review.py:
##########
@@ -100,6 +99,44 @@ def _get_map_index(q: str = Query("-1", alias="map_index"))
-> int:
MapIndexDep = Annotated[int, Depends(_get_map_index)]
+def _get_access_dependencies(*, method: str, airflow_v_3_1_plus: bool) ->
list[Any]:
+ """Return DAG access dependencies only for Airflow versions that support
HITL auth entities."""
+ if not airflow_v_3_1_plus:
Review Comment:
Why do you pass the bool as parameter in the function and do not use it in
the function body from the constant directly?
--
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]
