-
-
Notifications
You must be signed in to change notification settings - Fork 269
update theme #465
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 theme #465
Conversation
Hi @OriolAbril. So I've been learning a bit more about sphinx, but there's so much to it that I'm obviously not a master yet! Do you just need an approval for this? The built docs look fine with no problems. |
html_theme_options = { | ||
"icon_links": [ |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Why are you removing these?
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
answered in pymc-devs/pymc#6296 (comment)
@drbenvincent I would like some help double checking everything looks and works the same, that all links are working, things like that. After that leave a review with comments or aproval |
Where can we view it? |
Yes, alll 3 repos have a readthedocs "check", if you click on the "details" there you'll see the preview. For this PR the "details" link goes to https://pymcio--465.org.readthedocs.build/projects/examples/en/465/gallery.html |
Everything should be fixed now |
23184e7
to
213c647
Compare
3rd theme update PR, see pymc-devs/pymc#6296 for more details