Skip to content

Conversation

@LOCEANlloydizard
Copy link
Collaborator

Jupyter Book requires Node and was triggering an interactive prompt on RTD, causing EOFError. Adding nodejs: "22" should resolve the issue!

Jupyter Book requires Node and was triggering an interactive prompt on RTD, causing EOFError. Adding nodejs: "22" should resolve the issue!
Pin jupyter-book<0.16 so RTD uses the 0.15.x Sphinx-based CLI and supports config sphinx.
remove nodejs
@leewujung
Copy link
Member

The PR title doesn't seem to reflect the content?
Also looks like this works for echopop RTD build: https://github.com/OSOceanAcoustics/echopop/blob/main/.readthedocs.yaml
but it pins a weird jupyter book version...

@LOCEANlloydizard LOCEANlloydizard changed the title Fix RTD build by adding Node.js Pin jupyter-book to fix RTD build Dec 15, 2025
@LOCEANlloydizard
Copy link
Collaborator Author

Hi @leewujung, thanks! I’ve renamed the PR! The original name was linked to the first commit, but I later changed the approach... and aligned it with what Caesar did, as discussed here. I’ve also added an issue to this repo so we remember to unpin it (here)! Let me know if that works, if not I'll have a another look and compare with echopop way too!
Cheers!

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants