-
Notifications
You must be signed in to change notification settings - Fork 147
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
Automatically generate documentation for Server Process options #508
base: main
Are you sure you want to change the base?
Conversation
This will allow us to reuse it
Avoid duplication in documentation
Instead of writing the custom extension, we could also use |
I tried autodoc traits originally, but it's not suitable for generating https://jupyter-server-proxy.readthedocs.io/en/latest/server-process.html because although the underlying class uses Traitlets in this particular context we're not configuring it using Traitlets. Instead we're using a dict which is converted to the Traitlets object. |
Is there any particular reason we do not want to use |
Currently the documentation for
Server Process options
and the documentation in the code comments is redundant:jupyter-server-proxy/jupyter_server_proxy/config.py
Lines 168 to 283 in 11fcf89
jupyter-server-proxy/docs/source/server-process.md
Lines 20 to 242 in 11fcf89
This PR builds on top of #507 to autogenerate the documentation using a custom Sphinx Directive.