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.

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

DOC: Switch documentation from epydoc to pdoc #20

Open
wants to merge 5 commits into
base: master
Choose a base branch
from

Conversation

DWesl
Copy link
Contributor

@DWesl DWesl commented Aug 31, 2024

epydoc appears to be python-2-only. Since the extended support for Python 2 ended four years ago, it's probably time to update.

pdoc seems to have a similar design philosophy to epydoc, so I switched to that, rather than the more-configurable Sphinx.

GitHub Pages serves documentation from the docs directory rather than the html directory, so I changed the create_docs.sh script to put its output there. Let me know if you want me to revert that, or to rename the html directory to docs instead.

It should be possible to install the package required to do this with pip install /path/to/pyspharm[docs]

epydoc was never updated for python3.
pdoc seems to have a similar design philosophy,
although there are some differences from epytext.

I should ask about NumPy-style docstrings.
It would bring formatting into alignment,
but it would take more vertical space.
It would also look somewhat more natural in pydoc/help.
Google-style might make more sense with the existing choices,
but I don't know it as well.
@DWesl
Copy link
Contributor Author

DWesl commented Nov 9, 2024

Note: should change this to add mv docs/netCDF4/* docs/ to create_docs.sh as in netcdf4-python or add a redirect from docs/index.html to docs/netCDF4/index.html as suggested in the pdoc FAQ

EDIT: Apparently pdoc now adds the redirect itself if it's only passed one module, which simplifies things a bit.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

1 participant