-
-
Notifications
You must be signed in to change notification settings - Fork 560
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
update sphinxcontrib packages #39686
base: develop
Are you sure you want to change the base?
Conversation
sphinxcontrib-htmlhelp is taken care of on #39680. Would you mind making that a dependency and delete that upgrade here? |
I guess I can do that. When I have a bit of free time later today. |
Documentation preview for this PR (built with commit 02c7216; changes) is ready! 🎉 |
This PR does more than just the one package. I will update to take into account your PR. |
PR updated. |
Thanks very much! |
Looks good, thank you. |
sagemathgh-39686: update sphinxcontrib packages This is a follow up to sagemath#39577 where sphinx was update but not the sphinxcontrib packages. This causes breakages for some users. ### 📝 Checklist <!-- Put an `x` in all the boxes that apply. --> - [x] The title is concise and informative. - [x] The description explains in detail what this PR is about. - [x] 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 sagemath#39680 deals with one the packages that would be covered by this PR <!-- List all open PRs that this PR logically depends on. For example, --> <!-- - sagemath#12345: short description why this is a dependency --> <!-- - sagemath#34567: ... --> URL: sagemath#39686 Reported by: François Bissey Reviewer(s):
This is a follow up to #39577 where sphinx was update but not the sphinxcontrib packages.
This causes breakages for some users.
📝 Checklist
⌛ Dependencies
#39680 deals with one the packages that would be covered by this PR