-
Notifications
You must be signed in to change notification settings - Fork 21
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
release the pin on Sphinx version ? #83
Comments
Yes we could try updating the pin to |
I bumped into this while working on sphinx-gallery Jupyterlite integration since sphinx-gallery is testing for a range of sphinx-version up the sphinx development version. This is quite confusing right now that
will get you an old jupyterlite-sphinx version namely 0.4.0, which does not have the |
Yeah, this is because we did not have proper sphinx pinning at the time, we should have had the pin the entire time IMHO (not blaming anyone, I'm responsible for this I suppose). Let's release the pin then... Thanks for your PR! |
Yep, it does make sense given the constraints, but the behaviour for the user is still quite confusing. I guess this will be a lot better once there is a release with #85 in. In the meantime we can use a work-around. In my case it took me a while to realise that it was the root cause, I thought that somehow the sphinx events were not happening in the right order.
Just curious, would you mind saying a few words why? In my mind jupyterlite-sphinx is a library not an application and I would think it does not really need to pin its dependencies. Also the jupyterlite-sphinx, code does not seem to rely on too many sphinx internals so I would think that it is not that likely to break between Sphinx major releases. |
jupyterlite-sphinx 0.7.3 is released :) You can use it
Nothing prevents Sphinx devs to release a v7 that breaks the plugins API assumptions and break jupyterlite-sphinx. I want to prevent that, and to prevent the user to have to deal with debugging (like you had to do). But because we did not have proper pinning the entire time you ended up in this annoying position, sorry for that! |
Nice thanks a lot for this! It is true that my experience from working on sphinx-gallery a long time ago (things may have changed a bit since) was that Sphinx tended to break things a little too often in my biased opinion. |
In the current implementation, jupyterlite-shinx enforces a Sphinx 4 version. Sphinx 6 is out since December and Sphinx 4 is starting to be dropped by other Sphinx extensions.
Would it be possible to change/remove this pinned version from setup.py ?
The text was updated successfully, but these errors were encountered: