-
Notifications
You must be signed in to change notification settings - Fork 218
⬆️ Support Sphinx v9 #1076
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
base: master
Are you sure you want to change the base?
⬆️ Support Sphinx v9 #1076
Conversation
for more information, see https://pre-commit.ci
|
Currently, the major blocker appears to be that this function no longer seems to work properly: MyST-Parser/myst_parser/mdit_to_docutils/base.py Lines 1884 to 1929 in 59d5384
It leads to: Not sure what has changed to affect this yet, feedback / help welcome? |
|
@chrisjsewell Fairly sure this is a Docutils change rather than Sphinx. Notably Docutils doesn't seem to use Docutils currently uses the below code to add meta nodes: Could you do similar here? A |
@AA-Turner the only change in dependencies is sphinx 8.2.3 to 9.0.4 (plus the addition of roman-numerals==4.0.0), the docutils version remains the same at 0.21.2 (docutils is stilled pinned <0.22) is there anything you can think of that would change how docutils is initalised? |
@AA-Turner perhaps it is something to do with sphinx-doc/sphinx#13673 |
|
I noticed in the CI logs above, that the combination of Python 3.14 and Sphinx 9.0.4 does seem to pass on Windows for some reason: https://github.com/executablebooks/MyST-Parser/actions/runs/20259242759/job/58167505565?pr=1076 does this mean there might be some Python version and Sphinx interaction? |
It's not actually passing, if you look in the log, it's an issue in the windows job step, that I've seen before; it doesn't "fail-fast" like Linux and takes the fact that the |
Ah, well, makes sense. It did seem unusual that the failure would be platform dependent. So it's back to being a sphinx issue. |
|
|
@chrisjsewell I believe the error is fixed with sphinx-doc/sphinx#14194. However, I would strongly suggest not using A |
Thanks! will try it out soon
I didn't actually write this bit of code originally 😅 , so absolutely open to other solutions from you or anyone 👍
|
|
@AA-Turner |
|
What's the new error? A |
No description provided.