Skip to content

Doc: Add Function Documentation Style Guide#588

Merged
balos1 merged 3 commits intodevelopfrom
doc/function-doc-style
Oct 8, 2024
Merged

Doc: Add Function Documentation Style Guide#588
balos1 merged 3 commits intodevelopfrom
doc/function-doc-style

Conversation

@gardner48
Copy link
Copy Markdown
Member

@gardner48 gardner48 commented Oct 5, 2024

Some recent PRs have included comments about stylistic inconsistencies in the documentation. This PR adds a style guide for documenting user-callable and user-supplied functions to hopefully reduce this kind of traffic in future PRs.

@gardner48
Copy link
Copy Markdown
Member Author

Current PRs do not need to adopt this style but future additions to the docs should follow this guide. As time allows we can migrate existing documentation to this style.

Copy link
Copy Markdown
Collaborator

@drreynolds drreynolds left a comment

Choose a reason for hiding this comment

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

This looks great

Comment thread doc/superbuild/source/developers/style_guide/Documentation.rst Outdated
Comment thread doc/superbuild/source/developers/style_guide/Documentation.rst
Comment thread doc/superbuild/source/developers/style_guide/Documentation.rst Outdated
Co-authored-by: Daniel R. Reynolds <reynolds@smu.edu>
drreynolds
drreynolds previously approved these changes Oct 5, 2024
@balos1 balos1 added this to the SUNDIALS Next milestone Oct 7, 2024
@balos1 balos1 merged commit 1770cd7 into develop Oct 8, 2024
@balos1 balos1 deleted the doc/function-doc-style branch October 8, 2024 15:43
@gardner48 gardner48 mentioned this pull request Oct 18, 2024
gardner48 added a commit that referenced this pull request Jul 21, 2025
Some recent PRs have included comments about stylistic inconsistencies
in the documentation. This PR adds a style guide for documenting
user-callable and user-supplied functions to hopefully reduce this kind
of traffic in future PRs.

---------

Co-authored-by: Daniel R. Reynolds <reynolds@smu.edu>
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.

3 participants