Skip to content

Temporarily disable -W option on Sphinx build#6354

Merged
toshihikoyanase merged 1 commit intooptuna:masterfrom
c-bata:temp-fix-sphinx-warning
Nov 21, 2025
Merged

Temporarily disable -W option on Sphinx build#6354
toshihikoyanase merged 1 commit intooptuna:masterfrom
c-bata:temp-fix-sphinx-warning

Conversation

@c-bata
Copy link
Copy Markdown
Member

@c-bata c-bata commented Nov 21, 2025

Motivation

Currently, CI has been failed due to the following error:

WARNING: failed to reach any of the inventories with the following issues:
intersphinx inventory 'https://docs.python.org/3/objects.inv' not fetchable due to <class 'requests.exceptions.HTTPError'>: 503 Server Error: Service Unavailable for url: https://docs.python.org/3/objects.inv
https://github.com/optuna/optuna/actions/runs/19561736060/job/56015268613?pr=6317

Description of the changes

Disable -W option, so that sphinx does not treat warnings as errors.

@c-bata c-bata changed the title Temporarily disable -W option on Sphinx build Temporarily disable -W option on Sphinx build Nov 21, 2025
@c-bata c-bata marked this pull request as ready for review November 21, 2025 06:28
Copy link
Copy Markdown
Member

@toshihikoyanase toshihikoyanase left a comment

Choose a reason for hiding this comment

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

It seems a reasonable temporary fix. Thank you.

@toshihikoyanase toshihikoyanase merged commit 1f82a00 into optuna:master Nov 21, 2025
13 of 14 checks passed
@c-bata c-bata deleted the temp-fix-sphinx-warning branch November 21, 2025 06:32
c-bata added a commit to c-bata/optuna that referenced this pull request Dec 8, 2025
…rning"

This reverts commit 1f82a00, reversing
changes made to 25e478b.
not522 added a commit that referenced this pull request Dec 8, 2025
Revert PR #6354 to enable `-W` option on Sphinx build.
@not522 not522 added this to the v4.7.0 milestone Dec 17, 2025
@not522 not522 added the CI Continuous integration. label Jan 14, 2026
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

CI Continuous integration.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants