Skip to content
This repository has been archived by the owner on Dec 8, 2024. It is now read-only.

Setup Sphinx documentation #18

Merged
merged 8 commits into from
Aug 16, 2024
Merged

Setup Sphinx documentation #18

merged 8 commits into from
Aug 16, 2024

Conversation

LimaoC
Copy link
Owner

@LimaoC LimaoC commented Aug 16, 2024

Sets up Sphinx documentation (with the basic default theme for now) which will be autogenerated from the Python files in the repo, as well as a workflow that publishes it on GitHub pages whenever we push to main.

@LimaoC LimaoC requested a review from MitchellJC August 16, 2024 03:46
@LimaoC LimaoC self-assigned this Aug 16, 2024
@MitchellJC
Copy link
Collaborator

make html isn't working for me, are there some additional steps?

@LimaoC
Copy link
Owner Author

LimaoC commented Aug 16, 2024

make html isn't working for me, are there some additional steps?

Do you get the "sphinx-build not found" error? If so I think you might need poetry run make html (I'll update the README).

@MitchellJC
Copy link
Collaborator

make html isn't working for me, are there some additional steps?

Do you get the "sphinx-build not found" error? If so I think you might need poetry run make html (I'll update the README).

Yep that worked. I think we may need this Napolean extension for google docstrings? https://www.sphinx-doc.org/en/master/usage/extensions/napoleon.html

@LimaoC
Copy link
Owner Author

LimaoC commented Aug 16, 2024

Looks good, I'll add that in

@LimaoC LimaoC merged commit 05abf59 into main Aug 16, 2024
4 checks passed
@LimaoC LimaoC deleted the setup-sphinx branch August 16, 2024 04:58
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants