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.

Sign up for GitHub

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

Jump to bottom

build-docs workflow failed #5793

Closed
yiheng-wang-nv opened this issue Dec 30, 2022 · 2 comments · Fixed by #5794
Closed

build-docs workflow failed #5793

yiheng-wang-nv opened this issue Dec 30, 2022 · 2 comments · Fixed by #5794

Comments

@yiheng-wang-nv
Copy link
Contributor

The workflow failed in recent PRs such as: https://github.com/Project-MONAI/MONAI/actions/runs/3805194186/jobs/6472957035#step:7:243
@yiheng-wang-nv
Copy link
Contributor Author

it may due to sphinx-build new release: https://www.sphinx-doc.org/en/master/changes.html#release-6-0-0-released-dec-29-2022

@yiheng-wang-nv yiheng-wang-nv mentioned this issue Dec 30, 2022
Merged
7 tasks
@kbressem kbressem mentioned this issue Jan 1, 2023
Merged
7 tasks
@wyli
Copy link
Contributor

wyli commented Jan 4, 2023

looks like the issue here pydata/pydata-sphinx-theme#1094 I'll update the workaround in #5794

wyli added a commit to yiheng-wang-nv/MONAI that referenced this issue Jan 4, 2023
@wyli
adds workaround Project-MONAI#5793
4c2b0e9 
Signed-off-by: Wenqi Li <wenqil@nvidia.com>
@wyli wyli closed this as completed in #5794 Jan 4, 2023
wyli added a commit that referenced this issue Jan 4, 2023
@yiheng-wang-nv @pre-commit-ci @wyli
5793 Avoid using sphinx 6.0.0 (#5794)
e3d1cc8 
Fixes #5793 .

### Description

This PR provides a workaround for the incompatible sphinx 6.0.0

### Types of changes
<!--- Put an `x` in all the boxes that apply, and remove the not
applicable items -->
- [x] Non-breaking change (fix or new feature that would not break
existing functionality).
- [ ] Breaking change (fix or new feature that would cause existing
functionality to change).
- [ ] New tests added to cover the changes.
- [ ] Integration tests passed locally by running `./runtests.sh -f -u
--net --coverage`.
- [ ] Quick tests passed locally by running `./runtests.sh --quick
--unittests --disttests`.
- [ ] In-line docstrings updated.
- [ ] Documentation updated, tested `make html` command in the `docs/`
folder.

Signed-off-by: Yiheng Wang <vennw@nvidia.com>
Signed-off-by: Wenqi Li <wenqil@nvidia.com>
Co-authored-by: pre-commit-ci[bot] <66853113+pre-commit-ci[bot]@users.noreply.github.com>
Co-authored-by: Wenqi Li <wenqil@nvidia.com>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging a pull request may close this issue.

5793 Avoid using sphinx 6.0.0 yiheng-wang-nv/MONAI
2 participants
@wyli @yiheng-wang-nv