-
Notifications
You must be signed in to change notification settings - Fork 32
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
Rtd #187
Rtd #187
Conversation
Any objections to integrate this PR? |
Changes look good, so 👍 Is there a deploy of these docs anywhere yet (before this is merged)? Otherwise happy to go ahead and see how they look once merged. |
@will-moore doc available at https://jburel-ome-scripts.readthedocs.io/en/latest/ I had to push new commits since there were a few problems with Rtd itself |
This feels like a step towards being able to |
@will-moore happy with the status? |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Looks great, thanks!
Discussed with @sbesson and @will-moore this morning
We will keep the reference documentation in this repository
This PR replaces #183
docs/conf.py
to ensure that all scripts are listed in the technical documentation.