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

Update version of read the docs theme #468

Merged
merged 2 commits into from Aug 9, 2023

Conversation

Aurelienpujolm
Copy link
Contributor

@Aurelienpujolm Aurelienpujolm commented Aug 7, 2023

Checklist

  • I've given this PR a concise, self-descriptive, and meaningful title
  • I've linked relevant issues in the PR body
  • I've applied the relevant labels to this PR
  • I've added relevant tests for my contribution
  • I've updated the documentation and/or added correct docstrings
  • I've assigned a reviewer

Description

This PR is made to correct a bug caused by sphinx, because they removed the style argument in the newest version '7.0', I used this method to fix the issue (readthedocs/sphinx_rtd_theme#1465)

@Aurelienpujolm Aurelienpujolm self-assigned this Aug 7, 2023
@Aurelienpujolm Aurelienpujolm added the bug Something isn't working label Aug 7, 2023
@Aurelienpujolm
Copy link
Contributor Author

Not all tests will pass because of the bug in the master, this PR is just to correct the readthedocs.

Copy link
Member

@jcohenadad jcohenadad left a comment

Choose a reason for hiding this comment

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

This PR is make to correct a bug caused by sphinx, because they removed the style argument in the newest version '7.0', I used this method to fixed the issue (kivy/kivy#8230)

I understand this is a temporary fix, so can you please open an issue to address the 'long term' fix once it is fixed upstream by the sphinx/RTD team? thanks

@Aurelienpujolm
Copy link
Contributor Author

Aurelienpujolm commented Aug 9, 2023

This PR is make to correct a bug caused by sphinx, because they removed the style argument in the newest version '7.0', I used this method to fixed the issue (kivy/kivy#8230)

I understand this is a temporary fix, so can you please open an issue to address the 'long term' fix once it is fixed upstream by the sphinx/RTD team? thanks

I think that the newest method should always be good, because I'm now not just pinning a version of Sphinx, but I'm using the newest one for sphinx and sphinx_rtd_themes (as indicated in the newest link that I edited in the description of the PR).

If it's not enough, I will open a long term issue ! (I converted this PR to a draft one to give me some time to make sure that it's perfect)

@Aurelienpujolm Aurelienpujolm marked this pull request as draft August 9, 2023 14:41
@po09i po09i changed the title Changed version of Sphinx to be able to used style Update version of read the docs theme Aug 9, 2023
@po09i po09i changed the base branch from master to ad/dcm2bids-update August 9, 2023 16:21
@po09i po09i marked this pull request as ready for review August 9, 2023 16:23
@po09i po09i merged commit b5b5cc4 into ad/dcm2bids-update Aug 9, 2023
3 of 7 checks passed
@po09i po09i deleted the ap/edit-sphinx-version branch August 9, 2023 16:24
po09i added a commit that referenced this pull request Aug 9, 2023
* Update dcm2bids config fiel to follow 3.0.1 convention

* Update dcm2bids to version 3.0.1

* Update failing test

* Update version of read the docs theme (#468)

* Changed version of Sphinx to be able to used style

* Changed version of sphinx and sphinx theme

---------

Co-authored-by: Aurélien Pujol <63396133+Aurelienpujolm@users.noreply.github.com>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
bug Something isn't working
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

3 participants