potiuk commented on PR #35424: URL: https://github.com/apache/airflow/pull/35424#issuecomment-1793397165
BTW. I heavily extended this one and now it performs much more validation + it fixes the problem that security.rst was actually needlessly copied over in all providers. @utkarsharma2 @pankajastro @kaxil @pankajkoti -> this one should help (afte merging and rebasing) in making sure that the provider documentation is as expected for all the providers you developed. cc: @bolkedebruin -> we were talking about it when you added `common-io`. Rather than "templating" those separately - I added a pre-commit that will: a) generate missing documentation b) check if the index entries are added c) point out if executors are missing in provider.yaml (cc: @o-nikolas @vincbeck - it was missing for ECS but also it was missing for Kubernetes) d) if CLI-ref is defined, it will check that it is referred properly from Airlfow e) if configs are defined, it will check that they are referred properly from Airflow Genearally - that one will do all kinds of "let's check and generate consistent docs for new providers". -- 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]
