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.

Sign up for GitHub

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

Jump to bottom

Fix problems with the api docs on readthedocs #17

Merged
merged 1 commit into from Feb 3, 2019
Merged

Fix problems with the api docs on readthedocs #17

merged 1 commit into from Feb 3, 2019

Conversation

s-weigand
Copy link
Contributor

Adding this readthedocs.yml should fix the problem described in #14.

@coveralls
Copy link

coveralls commented Feb 3, 2019 •
edited

Pull Request Test Coverage Report for Build 37

  • 2 of 2 (100.0%) changed or added relevant lines in 2 files are covered.
  • No unchanged relevant lines lost coverage.
  • Overall coverage remained the same at 100.0%
Totals Coverage Status
Change from base Build 31: 0.0%
Covered Lines: 291
Relevant Lines: 291
💛 - Coveralls

oz123
oz123 requested changes Feb 3, 2019
View reviewed changes
Copy link
Owner

@oz123 oz123 left a comment

Choose a reason for hiding this comment

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

@s-weigand thank you for you ongoing efforts!
Can you please only include the commit with the yaml?

@s-weigand s-weigand force-pushed the fix-docs-on-rtd branch 2 times, most recently from 8491f39 to acb35ef Compare February 3, 2019 19:24
@s-weigand
Copy link
Contributor Author

done
I will make an extra PR for the flake8 problem

@oz123 oz123 merged commit c717f85 into oz123:master Feb 3, 2019
@s-weigand s-weigand mentioned this pull request Feb 3, 2019
Merged
@s-weigand s-weigand deleted the fix-docs-on-rtd branch February 8, 2019 16:51
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@oz123 oz123 oz123 approved these changes

Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.

None yet
3 participants
@s-weigand @coveralls @oz123