Skip to content

Conversation

@MMelQin
Copy link
Collaborator

@MMelQin MMelQin commented Nov 5, 2025

This PR updates Sphinx documentation dependencies and configuration to use newer versions and API standards to address the doc build issues on readthedoc. Local dev builds have been succeeding without issues.

The changes focus on upgrading Sphinx-related packages and updating deprecated myst-nb configuration options.

Key changes:

  • Updated Sphinx and related package versions with more explicit version constraints
  • Migrated myst-nb configuration from deprecated to current API (execution_in_temp → nb_execution_in_temp, jupyter_execute_notebooks → nb_execution_mode)
  • Removed redundant bibtex configuration section (already defined elsewhere in conf.py)

@MMelQin MMelQin requested a review from Copilot November 5, 2025 20:08
@sonarqubecloud
Copy link

sonarqubecloud bot commented Nov 5, 2025

Copy link
Contributor

Copilot AI left a comment

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 Sphinx documentation dependencies and configuration to use newer versions and API standards. The changes focus on upgrading Sphinx-related packages and updating deprecated myst-nb configuration options.

Key changes:

  • Updated Sphinx and related package versions with more explicit version constraints
  • Migrated myst-nb configuration from deprecated to current API (execution_in_tempnb_execution_in_temp, jupyter_execute_notebooksnb_execution_mode)
  • Removed redundant bibtex configuration section (already defined elsewhere in conf.py)

Reviewed Changes

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

File Description
docs/requirements.txt Updated version constraints for Sphinx (>=5.3.0,<8.0.0) and added minimum versions for sphinx-thebe and sphinx-autodoc-typehints
docs/source/conf.py Updated myst-nb configuration options to current API and removed duplicate bibtex configuration

💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.

@MMelQin MMelQin merged commit 0e36819 into main Nov 5, 2025
11 checks passed
@MMelQin MMelQin deleted the mq/readthedocs_fix branch November 5, 2025 21:29
MMelQin added a commit that referenced this pull request Nov 5, 2025
MMelQin added a commit that referenced this pull request Nov 5, 2025
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