-
Notifications
You must be signed in to change notification settings - Fork 44
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
ReadTheDocs configuration file (v2) #485
Conversation
Codecov Report
@@ Coverage Diff @@
## master #485 +/- ##
=======================================
Coverage 92.20% 92.20%
=======================================
Files 61 61
Lines 3246 3246
=======================================
Hits 2993 2993
Misses 253 253
Flags with carried forward coverage won't be shown. Click here to find out more. Continue to review full report at Codecov.
|
What was the decision about whether we should go ahead with this @CasperWA? Could we enable it for this PR and see how well it works? Pros:
Cons:
|
Just found that its already enabled (but not showing up in GitHub status checks), but the build is failing... |
The PR build itself is fine, so it should work in master too. I'm happy to merge this so we can evaluate both options "live", then eventually sunset the mkdocs/pages build and switch the URL over to readthedocs. Your call @CasperWA. |
Scratch that, |
Close #452
Move to ReadTheDocs. This PR adds a configuration file for it.