Skip to content
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

MAINT make sphinx conf more lenient towards missing jupyterlite #26705

Merged
merged 14 commits into from Jun 27, 2023

Conversation

glemaitre
Copy link
Member

@glemaitre glemaitre commented Jun 26, 2023

closes #26709

I am not really sure why but the min-doc build started to fail. It should be linked to #26627 because we bumped the version of sphinx to 6.0.0.

It would require to bump sphinx-gallery. To avoid bumping to the latest release 0.13 knows how to deal jupyterlite, we need to be lenient with the configuration and only add key in the conf if the extension is installed (otherwise, it fails locally).

@github-actions
Copy link

github-actions bot commented Jun 26, 2023

✔️ Linting Passed

All linting checks passed. Your pull request is in excellent shape! ☀️

Generated for commit: d59a16c. Link to the linter CI: here

doc/conf.py Outdated Show resolved Hide resolved
@lucyleeow
Copy link
Member

Ah could this be the cause of the failure: #26709

@glemaitre
Copy link
Member Author

Indeed, this is the same failure. Thanks for pointing out 0.10.1 because I was going to use 0.11.0 instedad.

@lesteve
Copy link
Member

lesteve commented Jun 27, 2023

doc-min-dependencies passes, merging, thanks!

@lesteve lesteve merged commit 371c921 into scikit-learn:main Jun 27, 2023
22 of 26 checks passed
jeremiedbb pushed a commit to jeremiedbb/scikit-learn that referenced this pull request Jun 29, 2023
punndcoder28 pushed a commit to punndcoder28/scikit-learn that referenced this pull request Jul 29, 2023
REDVM pushed a commit to REDVM/scikit-learn that referenced this pull request Nov 16, 2023
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.

MAINT: Sphinx gallery bug after Sphinx min dependency increased to 6.0.0
3 participants