Skip to content

[Docker] Bump CI image deps: sphinx-book-theme + z3-static#19835

Merged
tqchen merged 1 commit into
apache:mainfrom
tlopex:docker-deps-book-theme
Jun 19, 2026
Merged

[Docker] Bump CI image deps: sphinx-book-theme + z3-static#19835
tqchen merged 1 commit into
apache:mainfrom
tlopex:docker-deps-book-theme

Conversation

@tlopex

@tlopex tlopex commented Jun 18, 2026

Copy link
Copy Markdown
Member

This pr bumps the CI image Python dependencies, batched into one docker rebuild:

Docs theme (for the sphinx-book-theme migration):

  • ubuntu_install_sphinx.sh: replace sphinx_rtd_theme + tlcpack-sphinx-addon with sphinx-book-theme==1.1.4 + pydata-sphinx-theme==0.15.4. 1.1.4 (not 1.2.0) is used because it supports both Sphinx 8.1.3 (sphinx>=6.1, no upper bound) and the image's Python 3.10 (requires-python>=3.9); 1.2.0 requires Python>=3.11. 1.1.4 hard-pins pydata-sphinx-theme==0.15.4.
  • ubuntu_install_python_package.sh: drop the now-unused tlcpack-sphinx-addon pin.

Z3 prover:

  • ubuntu_install_python_package.sh: add z3-static==4.16.0.post1 for the optional Z3-backed Analyzer proving.

@gemini-code-assist gemini-code-assist Bot 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.

Code Review

This pull request updates the Sphinx documentation setup in the Docker installation scripts. Specifically, it removes the tlcpack-sphinx-addon package from both the general Python package installation and the Sphinx installation scripts. Additionally, it replaces the sphinx_rtd_theme with sphinx-book-theme and pydata-sphinx-theme in the Sphinx installation script. There are no review comments, and I have no feedback to provide.

Important

The consumer version of Gemini Code Assist on GitHub is being sunset. Starting June 18, 2026, new organization installations will be blocked, and all code review activity will officially cease on July 17, 2026.
For more details on the timeline and next steps, please review the Help Documentation.

Bump the CI image Python dependencies, batched into one docker rebuild:

Docs theme (for the sphinx-book-theme migration):
- ubuntu_install_sphinx.sh: replace sphinx_rtd_theme + tlcpack-sphinx-addon with
  sphinx-book-theme==1.1.4 + pydata-sphinx-theme==0.15.4. 1.1.4 (not 1.2.0) is
  used because it supports both Sphinx 8.1.3 (sphinx>=6.1, no upper bound) and the
  image's Python 3.10 (requires-python>=3.9); 1.2.0 requires Python>=3.11. 1.1.4
  hard-pins pydata-sphinx-theme==0.15.4.
- ubuntu_install_python_package.sh: drop the now-unused tlcpack-sphinx-addon pin.

Z3 prover:
- ubuntu_install_python_package.sh: add z3-static==4.16.0.post1 for the optional
  Z3-backed Analyzer proving.

This is the docker-bump step: after it merges and the rebuilt image is published,
ci/jenkins/docker-images.ini is pointed at the new tag, after which the
docs/conf.py book-theme switch (PR apache#19828) can build the docs.
@tlopex tlopex force-pushed the docker-deps-book-theme branch from 75dcfd2 to 3fba8aa Compare June 19, 2026 19:18
@tlopex tlopex changed the title [Docker] Switch docs image theme deps to sphinx-book-theme [Docker] Bump CI image deps: sphinx-book-theme + z3-static Jun 19, 2026
@tqchen tqchen merged commit 4174cdf into apache:main Jun 19, 2026
13 checks passed
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.

3 participants