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

Adds argument to doxygengroup to display just the description. #718

Merged

Conversation

intelkevinputnam
Copy link
Contributor

@intelkevinputnam intelkevinputnam commented Jul 27, 2021

Any doxygen directive that uses the _DoxygenContentBlockDirective can now display just the description, like so:

.. doxygengroup:: GROUPNAME
   :description-only:

I frequently build up groups from their components, so its useful to be able to display just the description.

Signed-off-by: Kevin Putnam kevin.putnam@intel.com

Copy link
Collaborator

@vermeeren vermeeren left a comment

Choose a reason for hiding this comment

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

@intelkevinputnam Could you also update documentation for doxygengroup and doxygennamespace directives?

  • documentation/source/directives.rst (arg only)
  • documentation/source/group.rst (arg + example)
  • documentation/source/namespace.rst (arg + example)

Doc also serves as tests and catching regressions via comparisons. Patch itself seems good to me.

Thanks!

Edit: You probably also want to rebase on master so CI works again.

breathe/directives.py Outdated Show resolved Hide resolved
@intelkevinputnam intelkevinputnam force-pushed the kp-add-description-only branch 2 times, most recently from f30078b to 2ae05ae Compare December 10, 2021 19:16
Signed-off-by: Kevin Putnam <kevin.putnam@intel.com>
Copy link
Collaborator

@vermeeren vermeeren left a comment

Choose a reason for hiding this comment

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

@intelkevinputnam Apologies for the long wait, the changes you made are great, many thanks for those!

Merging now.

michaeljones pushed a commit that referenced this pull request Feb 14, 2022
@michaeljones michaeljones merged commit 5162461 into breathe-doc:master Feb 14, 2022
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
code Source code enhancement Improvements, additions (also cosmetics)
Projects
None yet
Development

Successfully merging this pull request may close these issues.

Add ability to display just the description for any directive based on _DoxygenContentBlockDirective
3 participants