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

Document codebase #194

Merged
merged 11 commits into from
Oct 3, 2024
Merged

Document codebase #194

merged 11 commits into from
Oct 3, 2024

Conversation

Zeitsperre
Copy link
Collaborator

@Zeitsperre Zeitsperre commented Sep 25, 2024

Pull Request Checklist:

  • This PR addresses an already opened issue (for bug fixes / features)
    • This PR fixes #xyz
  • (If applicable) Documentation has been added / updated (for bug fixes / features).
  • (If applicable) Tests have been added.
  • CHANGES.rst has been updated (with summary of main changes).
    • Link to issue (:issue:number) and pull request (:pull:number) has been added.

What kind of change does this PR introduce?

  • Adds documentation to all public functions and class members of the code base.

Does this PR introduce a breaking change?

Yes. The DataBase class has been removed (never used it).

Other information:

This PR is using the standard gamut of checks offered via numpydoc (see: https://numpydoc.readthedocs.io/en/latest/validation.html). Eventually, we'll need to start exposing and documenting most of the hidden modules once we've determined what's safe to remove/migrate.

@Zeitsperre Zeitsperre added the documentation Improvements or additions to documentation label Sep 25, 2024
@Zeitsperre Zeitsperre self-assigned this Sep 25, 2024
@github-actions github-actions bot added the CI Changes to GitHub Actions and Tox label Sep 25, 2024
@Zeitsperre Zeitsperre marked this pull request as ready for review September 26, 2024 18:49
@github-actions github-actions bot added the docs label Sep 27, 2024
Copy link
Collaborator

@tlogan2000 tlogan2000 left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM

@Zeitsperre Zeitsperre merged commit d88f732 into cookiecutter-update Oct 3, 2024
25 of 26 checks passed
@Zeitsperre Zeitsperre deleted the document-code branch October 3, 2024 18:58
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
CI Changes to GitHub Actions and Tox docs documentation Improvements or additions to documentation
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants