Skip to content

Conversation

@kwankyu
Copy link
Collaborator

@kwankyu kwankyu commented Nov 7, 2025

Simple fix for https://groups.google.com/g/sage-release/c/x5zO6z5c0d4/m/CougOT6mBgAJ.

Simpler and non-destructive alternative to #41142

📝 Checklist

  • The title is concise and informative.
  • The description explains in detail what this PR is about.
  • I have linked a relevant issue or discussion.
  • I have created tests covering the changes.
  • I have updated the documentation and checked the documentation preview.

⌛ Dependencies

@kwankyu kwankyu marked this pull request as ready for review November 7, 2025 23:33
@kwankyu kwankyu requested a review from tobiasdiez November 7, 2025 23:34
@kwankyu kwankyu changed the title check jupytersphinx in doc build Restore check for jupytersphinx in doc build Nov 7, 2025
@dimpase
Copy link
Member

dimpase commented Nov 8, 2025

This is not tackling the root cause of the issue at hand, which is a broken venv with missing packages, forced to be broken by a configure hack.

@github-actions
Copy link

github-actions bot commented Nov 8, 2025

Documentation preview for this PR (built with commit de9a697; changes) is ready! 🎉
This preview will update shortly after each push to this PR.

@kwankyu
Copy link
Collaborator Author

kwankyu commented Nov 8, 2025

This PR, hopefully, fixes the direct cause of the issue at hand.

Let's wait for Marc's test.

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

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants