Skip to content

update sphinx version#3399

Merged
dlstadther merged 5 commits intospotify:masterfrom
hirosassa:update-sphinx
Feb 23, 2026
Merged

update sphinx version#3399
dlstadther merged 5 commits intospotify:masterfrom
hirosassa:update-sphinx

Conversation

@hirosassa
Copy link
Contributor

Description

SSIA

Motivation and Context

to follow latest sphinx package

Have you tested this? If so, how?

CI ran successfully

@hirosassa hirosassa requested review from a team and dlstadther as code owners February 22, 2026 14:35
Comment on lines -100 to -101
"Sphinx>=1.4.4,<2.0",
"sphinx_rtd_theme",
Copy link
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

support only Python 3.12 for building docs

Copy link
Collaborator

@dlstadther dlstadther left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

tests are failing + small comment

tox.ini Outdated
[testenv:docs]
# need Python 3.10
basepython = py310
# Python 3.12 required for Sphinx 9.x
Copy link
Collaborator

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

now that 3.13 support was added, feel free to bump this for 313

Copy link
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

fixed in d49c30d

Copy link
Collaborator

@dlstadther dlstadther left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

I'm only unsure of the RTD build. But we can't be 100% sure of that until it's merged and deployed.

@dlstadther dlstadther merged commit 0617778 into spotify:master Feb 23, 2026
57 checks passed
@hirosassa hirosassa deleted the update-sphinx branch February 23, 2026 13:53
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants