Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

[DO NOT MERGE] Verify sphinx-doc/sphinx#8115 fixes docs issue #1634

Closed
wants to merge 2 commits into from

Conversation

harupy
Copy link
Contributor

@harupy harupy commented Aug 14, 2020

Motivation

Just verify sphinx-doc/sphinx#8115 fixes the docs issue in sphinx>=3.1.0.

Description of the changes

Install sphinx on my branch that contains a fix patch and build the docs.

@harupy harupy changed the title [DO NOT MERGE] Verify sphinx-doc/sphinx#8115 fix the doc issue [DO NOT MERGE] Verify sphinx-doc/sphinx#8115 fixes docs issue Aug 14, 2020
@harupy
Copy link
Contributor Author

harupy commented Aug 14, 2020

Looks fine (the build-docs job doesn't fail & signatures of decorated classes look correct):

https://51980-122299416-gh.circle-artifacts.com/0/docs/build/html/reference/generated/optuna.integration.OptunaSearchCV.html?highlight=experimental

@harupy
Copy link
Contributor Author

harupy commented Aug 14, 2020

Hope other PRs (in the sphinx repo) won't get my fix messed up.

@harupy
Copy link
Contributor Author

harupy commented Aug 14, 2020

cc @crcrpar @hvy

@harupy harupy closed this Aug 14, 2020
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.

None yet

1 participant