pierrejeambrun commented on code in PR #57408:
URL: https://github.com/apache/airflow/pull/57408#discussion_r2472279526
##########
airflow-core/src/airflow/api_fastapi/common/parameters.py:
##########
@@ -186,7 +186,7 @@ def search_param_factory(
attribute: ColumnElement,
pattern_name: str,
skip_none: bool = True,
-) -> Callable[[str | None], _SearchParam]:
+) -> Callable[..., _SearchParam]:
Review Comment:
The callable returned is always `Callable[[str | None], _SearchParam]`, the
returned function is basically `depends_search` which is hardcoded and we don't
need to make it more generic at this point I think. Type was narrower before.
Same for other similar changes
--
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]
