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

Projects
None yet
2 participants
@rpgoldman
Copy link
Contributor

commented Jun 12, 2019

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

Robert P. Goldman

@rpgoldman rpgoldman merged commit a3fcc86 into pymc-devs:master Jun 12, 2019

1 check passed

continuous-integration/travis-ci/pr The Travis CI build passed
Details
@ColCarroll

This comment has been minimized.

Copy link
Member

commented Jun 12, 2019

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

@rpgoldman

This comment has been minimized.

Copy link
Contributor Author

commented Jun 12, 2019

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
You can’t perform that action at this time.