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

API Docs #24

Open
ZanSara opened this issue Apr 14, 2023 · 2 comments
Open

API Docs #24

ZanSara opened this issue Apr 14, 2023 · 2 comments

Comments

@ZanSara
Copy link
Contributor

ZanSara commented Apr 14, 2023

As discussed in #18, we could add auto-generated API docs to this project.

As done for Canals, I recommend using MkDocs (the Material theme is especially featureful) and host the result on GitHub Pages. It's generally very simple, and as long as the docstrings are well formatted with the Google style (see this SO answer) it's maintenance-free. Such docs can be fully automated with a simple GitHub workflow on every push to master.

MkDocs has great documentation, and in case of need an example of how to do create and deploy these API docs can be seen in Canals: the configuration lives in mkdocs.yml (made a bit complex by the mermaid support), the docs are found in the docs folder, and here is the workflow that deploys the docs.

@Mandera
Copy link
Contributor

Mandera commented Apr 16, 2023

Sounds great! A bit overwhelming since I've never tried it though.

I've been using the reST style docstrings. Is that something we could convert to Google style with black #23?

I do love mermaid so that sounds worth it!

Perhaps we'll only do this with generalimport to begin with, in a separate workflow file for now

@ZanSara
Copy link
Contributor Author

ZanSara commented Apr 17, 2023

I can totally help if you need! It was also the first time I used all of this in Canals and it was way more straightforward than I expected 🙂

I've been using the reST style docstrings. Is that something we could convert to Google style with black #23?

No, but you can probably use https://github.com/cbillingham/docconvert for the task. That would need to run just once and then once you commit the changes it should be good to go.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
Status: No status
Development

No branches or pull requests

2 participants