-
Notifications
You must be signed in to change notification settings - Fork 2
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
Adding documentation (Sphinx) and docstrings #164
Adding documentation (Sphinx) and docstrings #164
Conversation
Thanks @kallewesterling this looks great! I'll have a proper review at some point next week I think. In the meantime I'll use today's Seshat meeting to check on Majid's progress with merging the work on this fork so far to the upstream repo and updating the production site. Until that's done I've been avoiding editing parts of the code that aren't directly related to the maps, but I'm hopeful we'll be at that point very soon and can do things like this updating his docstrings etc and do more regular PRs to avoid large conflicts |
Hi @kallewesterling I've just had a go at checking this out locally and building the docs (and aim to look at the conflicts next). I've added
|
|
Add sphinx-rtd-theme
Ok nice, docs building for me now - will have a proper review tomorrow 👍 |
Great to hear!! Let me know if you run into any other problems! |
Tidy initial docs PR
I have opened a new issue #175 to ensure tests can be run by actions in future, but for now I have confirmed tests run locally so will merge |
This PR includes a lot of edits:
docs/requirements.txt
docs
folder (markdown) into new format (RST) in the newdocs
directory