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
First attempt at sphinx documentation #6
Conversation
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
important things to change:
- remove
requirements.txt
- add copyright year
with: | ||
python-version: '3.x' | ||
- name: Install Datasketches and Sphinx | ||
run: python -m pip install . sphinx==7.2.4 sphinx-rtd-theme |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
with luck we'll be able to remove the pinned version, or else use the fix in the issue about this: sphinx-doc/sphinx#11662 (comment)
``` | ||
|
||
## Problems | ||
The `density_sketch` and `tuple_sketch` are not yet included. |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
this seems to be an interaction with sphinx 7.2.5; 7.2.4 should work, and that's why it's pinned in the yaml.
we should probably look into including the packages in conf.py as a workaround, only because that seems like it should be mostly harmless.
add copyright year
This should automatically build the documentation for all sketches except density and tuple.
Please see the
docs/Readme.md
for further details.