-
Notifications
You must be signed in to change notification settings - Fork 252
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
Migrate to Readthedocs and Sphinx #890
Conversation
git push
Codecov ReportAttention: Patch coverage is
Additional details and impacted files@@ Coverage Diff @@
## master #890 +/- ##
==========================================
+ Coverage 52.29% 54.23% +1.93%
==========================================
Files 20 20
Lines 3991 4326 +335
==========================================
+ Hits 2087 2346 +259
- Misses 1904 1980 +76 ☔ View full report in Codecov by Sentry. |
You already know I find this absolutely incredible, Mark, and we discussed reviewing these things after merging. So on my end, we're good to go! |
@Joao-Dionisio thank you! I am extremely happy with the outcome of this. (you have to actually approve the MR which I always forget to do) |
This change moves our documentation from GitHub to Readthedocs.
I've added a basic structure / install guides / introduction to most plug-ins / motivation for using SCIP / FAQ / API reference manual.
I've also added a few new wrapper functions. These were needed when implementing the new examples in the documentation that are now duplicated as tests.
After this gets merged we'd need to do the following: