-
Notifications
You must be signed in to change notification settings - Fork 71
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
Documentation #39
Documentation #39
Conversation
Codecov Report
@@ Coverage Diff @@
## main #39 +/- ##
=========================================
Coverage 100.00% 100.00%
=========================================
Files 23 23
Lines 620 620
Branches 82 82
=========================================
Hits 620 620
Continue to review full report at Codecov.
|
I will merge this PR now, so that the deviation between the documentation and the actual code does not increase that much. |
You may want to consider using the Dask Sphinx theme:
https://github.com/dask/dask-sphinx-theme
…On Thu, Oct 1, 2020 at 12:52 PM Nils Braun ***@***.***> wrote:
Merged #39 <#39> into main.
—
You are receiving this because you are subscribed to this thread.
Reply to this email directly, view it on GitHub
<#39 (comment)>, or
unsubscribe
<https://github.com/notifications/unsubscribe-auth/AACKZTHQ5BRWFNURY3FZL6LSITMXDANCNFSM4RRFA6JQ>
.
|
* Improved documentation * Use the dask-sphinx-theme as mentioned in #39
Added documentation and readthedocs integration. Currently available at https://dask-sql.readthedocs.io/en/feature-documentation/index.html.
@kyprifog Would you mind having a look into the created documentation and tell me, if you think there is something missing? Would be very nice!