-
Notifications
You must be signed in to change notification settings - Fork 50
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
Replace instructions in step 2 with a pure markdown solution #296
Conversation
When I test this now, it does not seem to work. On stream it seemed fine, but maybe I did not reload the page? The directive is not recognized at all. When I replace ::: with ```, it is recognized by the api documentation page is not generated correcty. |
It seems we would need sphinx-autodoc2. See https://myst-parser.readthedocs.io/en/latest/syntax/code_and_apis.html#documenting-whole-apis |
Updated to use autodoc2. It's a bit different, but not really more complicated. Sphinx-autodoc2 would need to be added to the coderefinery environment for this to work. |
Co-authored-by: Ashwin V. Mohanan <[email protected]>
Marked ready for review, but we need to add sphinx-autodoc2 to the coderefinery environment |
But how about #293? Have you tried that one? I find it easier to use. Sorry I wasn't around in time to point it out earlier. |
Thank you for the explanation/comparison! |
Thank you for the work and wait! |
As in issue #295