-
Notifications
You must be signed in to change notification settings - Fork 22
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
Docs improvement #506
Docs improvement #506
Conversation
…o docs_improvement
a non-complete list of things to do to improve the documentation
a non-complete list of things to do to improve the documentation
…o docs_improvement
Nice, could already be merged or first we do those todo's |
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.
super classy, rigorous and with non-trivial catches and additional explanations.
Even includes a todo : ) Maybe go over it with Wout?
Really great. Should be merged asap and definitely before the next release.
sphinx>=5.3.0 # cannot be updated due to m2r2 no longer being maintained -> switch to MyST? |
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.
I know neither of the two, happy to switch to a maintained theme so we can keep sphinx up to date too yes!
ah, Wout already reviewed it. Do merge for users to immediately benefit, sphinx theme switch and additional todo's can go in future pull requests. |
m2r2 is not longer actively maintained, causing issues with newer versions of sphinx. As a bonus, cross-referencing is now easier, allowing to directly write links to .rst files instead of their build .html counterparts.
An attempt to fix some of the issues I found with the documentation