Skip to content

Rtd/sphinx compat - #7122

Merged
sklam merged 2 commits into
numba:masterfrom
esc:rtd/sphinx_compat
Jun 17, 2021
Merged

sklam merged 2 commits into
numba:masterfrom
esc:rtd/sphinx_compat

Conversation

@esc

@esc esc commented Jun 17, 2021 •

Copy link
Copy Markdown
Member

Should fix the failing RTD builds, inspired by:

sphinx-doc/sphinx#7747

esc added 2 commits June 17, 2021 13:36
@esc

esc commented Jun 17, 2021

Copy link
Copy Markdown
Member Author

@esc

esc commented Jun 17, 2021

Copy link
Copy Markdown
Member Author

@stuartarchibald

Copy link
Copy Markdown
Contributor

These type of URLs are working again! https://numba--7122.org.readthedocs.build/en/7122/

@stuartarchibald stuartarchibald left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

Thanks for the patch.

@stuartarchibald

Copy link
Copy Markdown
Contributor

Question... Does this create a minimum sphinx version requirement or is the patch backward compatible?

@stuartarchibald stuartarchibald added 5 - Ready to merge Review and testing done, is ready to merge and removed 3 - Ready for Review labels Jun 17, 2021
@stuartarchibald stuartarchibald added this to the Numba 0.54 RC milestone Jun 17, 2021
@stuartarchibald stuartarchibald added the Effort - short Short size effort needed label Jun 17, 2021
@esc

esc commented Jun 17, 2021

Copy link
Copy Markdown
Member Author

Screen Shot 2021-06-17 at 18 20 28

Deprecated since 1.8, removed with 4.0 -- yes, new minimum sphinx is now 4.0

@stuartarchibald

Copy link
Copy Markdown
Contributor
Screen Shot 2021-06-17 at 18 20 28

Deprecated since 1.8, removed with 4.0 -- yes, new minimum sphinx is now 4.0

Thanks, I guess we need to fix up the recipes and add a note in the docs too. Anything else?

@sklam
sklam merged commit 8dd28ab into numba:master Jun 17, 2021
@esc esc mentioned this pull request Jun 29, 2021
@esc
esc deleted the rtd/sphinx_compat branch July 20, 2021 11:00
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

5 - Ready to merge Review and testing done, is ready to merge Effort - short Short size effort needed

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants