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 build issues with ReadTheDocs #100

Merged
merged 8 commits into from
May 10, 2018
Merged

Fix build issues with ReadTheDocs #100

merged 8 commits into from
May 10, 2018

Conversation

philbull
Copy link
Collaborator

The docs are currently broken due to some import errors. This PR fixes that, and also fixes the formatting in a number of docstrings.
Current builds are being shown here:
http://hera-pspec.readthedocs.io/en/plot_docs/index.html

@philbull philbull requested a review from acliu May 10, 2018 21:33
@philbull philbull self-assigned this May 10, 2018
@philbull philbull added the bug label May 10, 2018
@philbull philbull added this to the Ready for IDR 2.1 analysis milestone May 10, 2018
@coveralls
Copy link

Coverage Status

Coverage increased (+0.02%) to 92.406% when pulling 5582a15 on plot_docs into 82c0a86 on master.

Copy link
Contributor

@acliu acliu left a comment

Choose a reason for hiding this comment

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

Looks great!

@philbull philbull merged commit 5dd299f into master May 10, 2018
@philbull philbull deleted the plot_docs branch May 10, 2018 21:49
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants