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-mdinclude to 0.5.4 #6145

Merged
merged 1 commit into from
Apr 23, 2024

Conversation

eddyashton
Copy link
Member

A new week, a new transitive-dependency-with-imprecise-version-constraints-causing-our-build-to-break. Our GitHub Pages action broke in the past few days:

Could not import extension sphinxcontrib.openapi (exception: cannot import name 'error_string' from 'docutils.io' (/data/src/2.CCF/env/lib/python3.8/site-packages/docutils/io.py))

Spelunking through packages named in the errors showed no recent releases, but a full code search showed sphinx-mdinclude (required by sphinxcontrib.openapi) using docutils.io, specifically:

from docutils.io import error_string as ErrorString

in v6.0, which was released a few days ago. Have pinned to previous version for now.

@eddyashton eddyashton requested a review from a team as a code owner April 23, 2024 08:06
@achamayou achamayou merged commit fca8827 into microsoft:main Apr 23, 2024
14 checks passed
@achamayou achamayou added auto-backport Automatically backport this PR to LTS branch 4.x-todo PRs which should be backported to 4.x labels May 1, 2024
ccf-bot pushed a commit that referenced this pull request May 1, 2024
(cherry picked from commit fca8827)
@ccf-bot ccf-bot added the backported This PR was successfully backported to LTS branch label May 1, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
4.x-todo PRs which should be backported to 4.x auto-backport Automatically backport this PR to LTS branch backported This PR was successfully backported to LTS branch
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

3 participants