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

Autosection #164

Merged
merged 3 commits into from
Jul 4, 2023
Merged

Autosection #164

merged 3 commits into from
Jul 4, 2023

Conversation

jnikula
Copy link
Owner

@jnikula jnikula commented Jul 2, 2023

#161 with c:autosection and cpp:autosection naming, and some tweaks on the documentation/examples, but no real functional changes.

With the introduction of directives for extracting documentation for
individual symbols, documentation comments not related to any symbols
were slightly neglected. The only way to grab them is to use c:autodoc
or cpp:autodoc.

Introduce directives for extracting documentation for individual text
documentation comments, or "top level comments" as referred to in the
parser.

The obvious downside is having to add means to reference the comments by
name, which means parsing documentation comments. To keep it simple, the
name shall be from the first alphanumeric character in the comment,
inclusive, to the next :, ., or newline, non-inclusive, with no further
parsing.

Note that this does *not* automatically generate any hyperlink targets
or Sphinx domain directives.

Use c:autosection and cpp:autosection naming with the idea of extending
the directives with the :members: option later to include a section of
documentation between top level comments.
@BrunoMSantos
Copy link
Collaborator

Looks good to me ;)

Created #166 so we remember the members stuff.

@jnikula jnikula merged commit 9a29e25 into master Jul 4, 2023
4 checks passed
@jnikula jnikula deleted the autosection branch July 4, 2023 08:58
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.

None yet

2 participants