Skip to content

Start Executing Docs #486

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

Open
wants to merge 2 commits into
base: develop
Choose a base branch
from
Open

Conversation

ianhi
Copy link

@ianhi ianhi commented Mar 12, 2025

Beginning of executable docs (#83). All notebooks will automatically be executed as part of the docs build per the docs here: https://myst-nb.readthedocs.io/en/latest/computation/execute.html#

It will be useful to set up the build failing on unexepcted errors per the docs here: https://myst-nb.readthedocs.io/en/latest/computation/execute.html#error-reporting-warning-vs-failure

Currently has multiple errors in the usage.ipynb file @TomNicholas

ianhi added 2 commits March 12, 2025 15:54
@TomNicholas TomNicholas added the documentation Improvements or additions to documentation label Mar 12, 2025
@TomNicholas
Copy link
Member

This is great, thank you @ianhi !

Can you just explain how (if at all) this interacts with our CI? I see you can locally configure to automatically rebuild the docs but will these executable docs automatically get rerun by our docs CI? Or should we add that in a follow-up?

@ianhi
Copy link
Author

ianhi commented Mar 12, 2025

The readthedocs build will fail if there is an error, then when you push an update (that hopefully resolves things!) it will run again.

@TomNicholas TomNicholas mentioned this pull request Mar 17, 2025
6 tasks
@maxrjones
Copy link
Member

thanks for working on this @ianhi! I wanted to let you know of a couple necessary changes related to #407 (sorry for the extra work):

Is there a way to execute the markdown code snippets directly rather than converting all the markdown files to Jupyter notebooks?

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
documentation Improvements or additions to documentation
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants