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

Pin Sphinx 3.x to docutils <0.17 #9071

Merged
merged 1 commit into from
Apr 9, 2021

Conversation

ericholscher
Copy link
Contributor

@ericholscher ericholscher commented Apr 8, 2021

This will address the issues that the latest docutils release caused.
I think this is a good practice in general to make sure we have a defined range of docutils versions,
given that they might change in the future.
Having this defined will mean that 3.x versions of Sphinx will always work,
even when docutils has advanced with additional backwards incompatible features.

We've had more than 10 support requests about this in the past week, so I think this is hitting a lot of users throughout the Sphinx ecosystem and it's very hard to debug if you don't understand what is happening.

Refs #9065 #9063 #9061 #9051

@ericholscher ericholscher changed the base branch from master to 3.x April 8, 2021 21:45
@tk0miya
Copy link
Member

tk0miya commented Apr 9, 2021

Could you rebase this into 3.5.x branch? Then I'll merge this soon.

Note: 3.5.4 will be the last release of 3.x series. So I will merge this to the package. But I will not merge this into 4.x series because it should go forward. No reason to restrict the version.

This will address the issues that the latest docutils release caused.
I think thi si a good practice in general to make sure we have a defined range of docutils versions,
given that they might change in the future.
Having this defined will mean that 3.x versions of Sphinx will always work,
even when docutils has advanced with additional backwards incompatible features.

Refs sphinx-doc#9065 sphinx-doc#9063 sphinx-doc#9061 sphinx-doc#9051
@ericholscher ericholscher changed the base branch from 3.x to 3.5.x April 9, 2021 15:33
@ericholscher
Copy link
Contributor Author

@tk0miya Done 👍

@ericholscher
Copy link
Contributor Author

I do think that specifically defining a upper bounds of docutils support in Sphinx is a good ongoing policy, but that's a larger discussion. It certainly has downsides, but it could be useful to have Sphinx increase the docutils dependency after it's been tested against the currently released version. Currently I don't believe docutils is testing Sphinx in it's releases, so it seems we'll hit this issue again whenever docutils makes a useful but breaking change.

@tk0miya tk0miya merged commit eb94406 into sphinx-doc:3.5.x Apr 9, 2021
@tk0miya
Copy link
Member

tk0miya commented Apr 9, 2021

Thanks!

tk0miya added a commit that referenced this pull request Apr 10, 2021
@tk0miya tk0miya mentioned this pull request Apr 10, 2021
@github-actions github-actions bot locked as resolved and limited conversation to collaborators Jul 12, 2021
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

2 participants