potiuk commented on code in PR #36311:
URL: https://github.com/apache/airflow/pull/36311#discussion_r1440484931
##########
docs/exts/operators_and_hooks_ref.py:
##########
@@ -215,6 +215,136 @@ def _render_deferrable_operator_content(*,
header_separator: str):
return _render_template("deferrable_operators_list.rst.jinja2",
providers=providers)
+def _get_public_classes(module_filepath: str, class_extras: dict[str, Any]) ->
dict[str, dict[str, Any]]:
Review Comment:
Yeah I think it's far easier to reason and tweak those kind of functionality
in our code rather than relying on internals of Sphinx or Auto API - we are
stuck with old versions of those and there is an on-going effort on migrating
to newer versions - any couplling with internals there will make the migration
efforr more dificult unless we know and can rely on Sphinx APIs.
Neither Sphinx nor AutoAPI have been designed to provide such features to
other users - it's not their purpose, they are tools to do stuff not libraries
to provide such functionality so I very much prefer to have our own code rather
than rely on their code.
Assume this question is resolved.
--
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]
