potiuk commented on PR #34921: URL: https://github.com/apache/airflow/pull/34921#issuecomment-1793427809
> @utkarsharma2 we have `security.rst` in other provider docs example: https://github.com/apache/airflow/blob/main/docs/apache-airflow-providers-apache-cassandra/security.rst do we need to add this here too @pankajastro @utkarsharma2 FYI. This is exactly the kind of problems that #35424 is going to solve (and I implemented it looking at your struggles here :) ) After #35424 is merged, all such missing bits will be caught (and most of them fixed automatically at pre-commit time. Also, the `security.rst` got a lot simpler in #35435 - just one line of include. The Pre-commit in #35424 will even generate (and update) the files automatically for you, and it will tell you if you have not added the links in indexe explicitly at pre-comit time So I'd say - make sure that all the LLM providers are rebased and pre-commits run after we merged #35424 - this will not only verify if all docs are properly structured, but also will correct any inconsistencies it fints. -- 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]
