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

Add target to validate docstrings with numpydoc #86

Open
wants to merge 9 commits into
base: main
Choose a base branch
from

Conversation

santisoler
Copy link
Member

@santisoler santisoler commented May 21, 2024

Add new target to the Makefile to validate docstrings using numpydoc. Add rules for validating docstrings using numpydoc to pyproject.toml. Run make numpydoc in GitHub Actions. Fix some of the errors that numpydoc raised: add description to functions listed in "See also" sections; sort "See also", "Notes" and "References" sections in the right order; add missing docstring to modules; add missing description of parameters and returns to private functions.

Add rules for validating docstrings using `numpydoc` to
`pyproject.toml`.
Add new target to the Makefile to validate docstrings using numpydoc.
This allows to use the `numpydoc lint` command instead of launching
a Python module from the terminal.
@leouieda
Copy link
Member

leouieda commented Dec 2, 2024

This is great!

Add missing comments to the objects listed in See Also in the magnetic
functions for prisms, and fix the order of the sections (See Also,
Notes, References).
@santisoler santisoler marked this pull request as ready for review December 6, 2024 23:47
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.

2 participants