dilnazanlid commented on code in PR #72474:
URL: https://github.com/apache/airflow/pull/72474#discussion_r4102991621
##########
airflow-core/src/airflow/api_fastapi/core_api/datamodels/dag_warning.py:
##########
@@ -30,7 +30,7 @@ class DAGWarningResponse(BaseModel):
"""Dag Warning serializer for responses."""
dag_id: str
- warning_type: DagWarningType
+ warning_type: DagWarningType | str
Review Comment:
I kept DagWarningType as a plain enum, because putting the anyOf on the enum
schema turned it into a wrapper in generated clients and broke
DagWarningType.<MEMBER> (airflow-ctl needed a test fix). Importer types are now
a separate namespaced ImporterWarningType, combined with the enum in one
DagWarningTypeValue alias that the response, the filter and the model all use,
so it's still a single definition.
--
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]
