potiuk commented on code in PR #47171:
URL: https://github.com/apache/airflow/pull/47171#discussion_r1975477974
##########
clients/python/openapi_v1.yaml:
##########
Review Comment:
I think so, it's somewhat remnant of the distant past where the client was
generated in a separate repo so it had to be copied there - but likey we could
use it from other place in the repo now as it is done inside the monorepo, I
think the only potential problem is that the python client's package currently
contains the v1.yaml as an asset inside (just for posterity ) so we would have
to make sure it's still there (or decide not to do it):
```
⌂9.28 [jarek:~/code/airflow]
better-documentation-about-updating-depenendencies+ 59s ± unzip -t
dist/apache_airflow_client-2.10.0-py3-none-any.whl
Archive: dist/apache_airflow_client-2.10.0-py3-none-any.whl
testing: v1.yaml OK
testing: airflow_client/__init__.py OK
testing: airflow_client/client/__init__.py OK
testing: airflow_client/client/api_client.py OK
testing: airflow_client/client/configuration.py OK
testing: airflow_client/client/exceptions.py OK
testing: airflow_client/client/model_utils.py OK
testing: airflow_client/client/rest.py OK
```
--
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]
