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

Fix minor bug in the Sphinx configuration. #3519

Merged
merged 1 commit into from Jun 12, 2019

Conversation

rpgoldman
Copy link
Contributor

Warning revealed that the value of html_theme_path was of the wrong type.

@rpgoldman rpgoldman merged commit a3fcc86 into pymc-devs:master Jun 12, 2019
@ColCarroll
Copy link
Member

To be clear, this change suppresses the warning, but doesn't change how the docs look, right?

@rpgoldman
Copy link
Contributor Author

To be clear, this change suppresses the warning, but doesn't change how the docs look, right?

Yes, that's correct. TBH, I don't know how it worked before, because as far as I could tell, sphinx couldn't find that style directory. But it works now.

I think we might be happier with a themes/ subdirectory with semantic-sphinx contained in it, but that probably isn't really necessary unless we start using more than a single theme.

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

2 participants