Skip to content

MAINT: Replace build_sphinx with sphinx-build#1139

Merged
larsoner merged 1 commit intosphinx-gallery:masterfrom
oscargus:sphinxbuild
May 15, 2023
Merged

MAINT: Replace build_sphinx with sphinx-build#1139
larsoner merged 1 commit intosphinx-gallery:masterfrom
oscargus:sphinxbuild

Conversation

@oscargus
Copy link
Copy Markdown
Contributor

@oscargus oscargus commented May 13, 2023

In Sphinx 7.0.1 support for build_sphinx was removed. I think this is the way to go.

Also installs imageio as I think that is not an expected failure...

Edit: install absl-py for Sphinx-dev test. Also, sphinx-dev adds ?hash(?) to href-link so removed the closing " from the test.

@oscargus oscargus force-pushed the sphinxbuild branch 4 times, most recently from e6d2049 to 6bb16ea Compare May 13, 2023 11:36
@oscargus oscargus changed the title Replace build_sphinx with sphinx-build MAINT: Replace build_sphinx with sphinx-build May 13, 2023
Copy link
Copy Markdown
Contributor

@lucyleeow lucyleeow left a comment

Choose a reason for hiding this comment

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

Thanks for the fix, just a few questions

@lucyleeow
Copy link
Copy Markdown
Contributor

In Sphinx 7.0.1 support for build_sphinx was removed. I think this is the way to go.

For posterity: sphinx-doc/sphinx#9595

@larsoner larsoner merged commit 830b57f into sphinx-gallery:master May 15, 2023
@larsoner
Copy link
Copy Markdown
Contributor

Thanks @oscargus , and @lucyleeow ignore my ping from the other PR, sorry I wasn't following properly! 🤦

@oscargus oscargus deleted the sphinxbuild branch May 15, 2023 19:08
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants