Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Add never_run_doxygen and never_run_sphinx_apidoc options #159

Merged
merged 2 commits into from
Nov 21, 2024

Conversation

rkent
Copy link
Contributor

@rkent rkent commented Nov 18, 2024

Some large packages generate API documentation themselves. These same packages often take large amounts of time in rosdoc2 runs, with output that is not useful since the documentation options are often not optimal compared to the externally generated documentation.

This PR allows rosdoc2.yaml options to disable attempts by rosdoc2 to generate API docs in those cases.

An example is pinocchio which has complete documentation, including API documentation, at https://gepettoweb.laas.fr/doc/stack-of-tasks/pinocchio/master/doxygen-html/

@rkent rkent requested review from audrow and tfoote as code owners November 18, 2024 20:53
Copy link
Member

@tfoote tfoote left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Adding these options makes sense. One request to improve the debug messages.

Copy link
Member

@tfoote tfoote left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Thanks for the iteration. It looks good to me.

@tfoote tfoote merged commit 99adc39 into ros-infrastructure:main Nov 21, 2024
5 checks passed
@rkent rkent deleted the pr-support-never-run-options branch December 10, 2024 16:09
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants