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

Use sphinx rst format in all docstrings #18

Merged
merged 1 commit into from
Jan 14, 2023
Merged

Conversation

ItsDrike
Copy link
Member

@ItsDrike ItsDrike commented Jan 13, 2023

This converts all docstrings in the library to use sphinx style, getting the codebase ready for auto-generated docs, that pick these docstrings up.

On top of just conversion to a different format, this does also make some general improvements to the wording in some places, and perhaps even adds new docstrings for undocumented things.

  • Convert all docstrings in the project to use sphinx style
  • Also convert docstrings in tests/, even though these won't be in docs, we want consistency

This PR is a part of #19.

@ItsDrike ItsDrike added a: documentation Related to project's documentation (comments, docstrings, docs) t: enhancement Improvement to an existing feature p: 1 - high This should be addressed quickly labels Jan 13, 2023
@ItsDrike ItsDrike force-pushed the sphinx-format-docstrings branch from f9de505 to 074cb78 Compare January 13, 2023 23:16
@ItsDrike ItsDrike changed the base branch from add-docs to main January 13, 2023 23:16
@ItsDrike ItsDrike mentioned this pull request Jan 13, 2023
23 tasks
@ItsDrike ItsDrike force-pushed the sphinx-format-docstrings branch from 074cb78 to a3b01da Compare January 14, 2023 00:07
@ItsDrike ItsDrike force-pushed the sphinx-format-docstrings branch from a3b01da to 6e2be0d Compare January 14, 2023 21:13
@ItsDrike ItsDrike merged commit 3348dcf into main Jan 14, 2023
@ItsDrike ItsDrike deleted the sphinx-format-docstrings branch January 14, 2023 21:16
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
a: documentation Related to project's documentation (comments, docstrings, docs) p: 1 - high This should be addressed quickly t: enhancement Improvement to an existing feature
Projects
None yet
Development

Successfully merging this pull request may close these issues.

1 participant