ome / omero-scripts

Core OMERO Scripts
https://pypi.org/project/omero-scripts/
12 stars 32 forks source link

Docs on readthedocs #183

Closed jburel closed 3 years ago

jburel commented 3 years ago

This PR follows up on some work started by @will-moore It publishes the doc on readthedocs. see https://jburel-omero-scripts.readthedocs.io/en/latest/

The link in the README under Usage is not working. This is the suggestion for the final link we could use "ome" too.

Improve layout in readme too see https://github.com/jburel/scripts/tree/docs_sphinx

To test locally c53d456

joshmoore commented 3 years ago

Screen Shot 2021-02-22 at 08 51 41

certainly looks nice, J-m! Did you notice whether or not there's a warning/error if a script is added but not linked in the docs?

jburel commented 3 years ago

There is no warning or error by default. This will require some code in config.py to scan the scripts directory and compare with the list of entries with automodule

jburel commented 3 years ago

if the script is listed using "automodule" and does not exist, there is a warning

WARNING: autodoc: failed to import module 'Kymograph'; the following exception was raised:
No module named 'Kymograph'

So I need to add a check for the situation when we have a script but no entry in the doc.

jburel commented 3 years ago

The readthedocs webhook will build when a PR is merged into master that will become the "latest" as it stands For the url, we can use either the canonical or a domain specific. It might be easier to the canonical especially if we roll out that strategy across the various repositories

jburel commented 3 years ago

see https://docs.readthedocs.io/en/stable/versions.html

jburel commented 3 years ago

Replaced by https://github.com/ome/omero-scripts/pull/187