Skip to content

Use Python 3.11 instead of 3.10 in the doc builiding - #895

Draft
AVHopp wants to merge 3 commits into
mainfrom
docs/bump_to_python311_new
Draft

Use Python 3.11 instead of 3.10 in the doc builiding#895
AVHopp wants to merge 3 commits into
mainfrom
docs/bump_to_python311_new

Conversation

@AVHopp

@AVHopp AVHopp commented Aug 18, 2026

Copy link
Copy Markdown
Collaborator

This PR bumps the python version used in the doc building jobs to 3.11.

This is necessary due to the way that our guarded forward references are handled. Since doc building and link checking are separateed now, the actual doc building does not have access to already "compiled" files. As a consequence, it cannot check and verify that the references can actually resolve. We thus need to ignore errors coming from those references. However, the corresponding option is only available in a newer version of sphinx-apidoc-typehints which is not available on python 3.10 and hence requires us to update to python 3.11.


Stack created with GitHub Stacks CLIGive Feedback 💬

Copilot AI lite review requested due to automatic review settings August 18, 2026 15:06
@AVHopp AVHopp changed the title Use Python 3.11 instead of 3.10 in the doc builiding - #892 Use Python 3.11 instead of 3.10 in the doc builiding Aug 18, 2026

Copilot AI 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.

Pull request overview

This PR updates the documentation build jobs to run on Python 3.11 instead of Python 3.10, aligning CI/doc workflows with the newer sphinx-apidoc-typehints support needed for the current docs build/linkcheck separation.

Changes:

  • Renamed the tox docs environment from docs-py310 to docs-py311.
  • Updated GitHub Actions workflows (ci, regular, docs) to set up Python 3.11 and invoke tox -e docs-py311.
  • Added a changelog entry documenting the docs build Python version bump.

Reviewed changes

Copilot reviewed 5 out of 5 changed files in this pull request and generated no comments.

Show a summary per file
File Description
tox.ini Renames the docs tox env to docs-py311 to match the new docs-build interpreter target.
CHANGELOG.md Notes the switch to Python 3.11 for building documentation.
.github/workflows/regular.yml Updates the scheduled/regular docs job to use Python 3.11 and docs-py311.
.github/workflows/docs.yml Updates the docs workflow to use Python 3.11 and docs-py311.
.github/workflows/ci.yml Updates the CI docs build step to use Python 3.11 and docs-py311.

💡 Add a code-review agent skill or configure MCP servers for context-aware, tailored reviews. Learn more in the docs.

Comment thread CHANGELOG.md Outdated
AVHopp added 2 commits August 19, 2026 09:02
This is necessary as we need to update one of the sphinx
related packages for suppressing some of the errors that
are now surfacing due to the separated build of the
doc building and the link checking
@AVHopp
AVHopp force-pushed the docs/bump_to_python311_new branch from b538473 to 92152d3 Compare August 19, 2026 07:08
Co-authored-by: Martin Fitzner <martin.fitzner@merckgroup.com>
Comment thread .github/workflows/ci.yml
@AVHopp
AVHopp marked this pull request as draft August 20, 2026 08:56
@AVHopp
AVHopp marked this pull request as ready for review August 20, 2026 10:47
Comment thread .github/workflows/ci.yml

Copy link
Copy Markdown
Collaborator Author

Choose a reason for hiding this comment

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

@Scienfitz @AdrianSosic it might (not sure yet) turn out that we do not need this PR as some of the things related to linkchecking might change, we'll probably need to discuss this. Do we want to update the python version for doc building anyway or only if necessary?

Copy link
Copy Markdown
Collaborator

Choose a reason for hiding this comment

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

if its not needed and no other reason for the upgrade transpires it shouldn't be done

@AVHopp
AVHopp marked this pull request as draft August 20, 2026 13:31
@AVHopp AVHopp self-assigned this Aug 20, 2026
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