fat-catTW commented on code in PR #63624: URL: https://github.com/apache/airflow/pull/63624#discussion_r2975040004
########## airflow-core/src/airflow/api_fastapi/common/parameters.py: ########## @@ -785,6 +787,46 @@ def depends( QueryAssetDependencyFilter = Annotated[_AssetDependencyFilter, Depends(_AssetDependencyFilter.depends)] +class _ConsumingAssetFilter(BaseParam[str | None]): + """Filter DAG runs by consuming asset (name or URI).""" + + def to_orm(self, select: Select) -> Select: + if self.value is None and self.skip_none: Review Comment: How about we use: if self.value in (None, "") and self.skip_none: It ensures the condition is more explicit. -- 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]
