jpmarques66 commented on a change in pull request #10351:
URL: https://github.com/apache/airflow/pull/10351#discussion_r481175895
##########
File path: airflow/api_connexion/openapi/v1.yaml
##########
@@ -1447,8 +1447,9 @@ components:
readOnly: true
nullable: true
state:
- $ref: '#/components/schemas/DagState'
readOnly: True
+ oneOf:
+ - $ref: '#/components/schemas/DagState'
Review comment:
@houqp So that was the idea in the beginning since spectral started
complaining about that (having siblings of $ref). The
[docs](https://swagger.io/docs/specification/using-ref/) refer that siblings
are ignored, but it seems that the code generation isn't prepared for this.
I'm still a bit unfamiliar with the project, do you think this should be
reverted?
----------------------------------------------------------------
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.
For queries about this service, please contact Infrastructure at:
[email protected]
