-
Notifications
You must be signed in to change notification settings - Fork 129
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
Developer docs #361
Developer docs #361
Conversation
…nswers-instructions
Do the docs in each PR get built automatically? If so, could we add the ReadTheDocs URL for this PR as a comment here? |
Yes, for this branch I told RTD to build the docs automatically. Here is the link https://sundials.readthedocs.io/en/feature-udpating-answers-instructions/ |
GitLab CI is reporting as failed since Corona is (or was) down. I will rerun the jobs. It would be nice to merge this PR before the removal of deprecated code PR @gardner48. |
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.
Minor question about the submodule remote but otherwise this looks good.
We'll need to make some updates for displaying graphviz figures on the readthedocs page (or it seems that way on my phone) but that's a separate issue.
Add developer docs to RTD superbuild Add instructions for updating answers in the developer docs. --------- Co-authored-by: David Gardner <[email protected]>
Add developer docs to RTD superbuild Add instructions for updating answers in the developer docs. --------- Co-authored-by: David Gardner <[email protected]>
Also adds instructions for updating answers in the developer docs.
RTD preview: https://sundials.readthedocs.io/en/feature-udpating-answers-instructions/