Skip to content
This repository was archived by the owner on Nov 10, 2025. It is now read-only.

Conversation

@guicho271828
Copy link

Related Issue

Supports #ISSUE_NUMBER

Related PRs

This PR is not dependent on any other PR

What this PR does / why we need it

I simply updated the template docstring.

Although this PR may not seem important, it could have a significant effect of encouraging the authors to write a proper docstring; The original docstrings, e.g. """Base Class for all Generators""", are not clearly stating that they originate from the template. In constrast, the new docstring clearly conveys the message """TODO: Copy and edit this template to implement your own validator class""".

Special notes for your reviewer

If applicable**

  • this PR contains documentation
  • this PR contains unit tests
  • this PR has been tested for backwards compatibility

Copy link
Collaborator

@mvcrouse mvcrouse left a comment

Choose a reason for hiding this comment

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

This is good, I've been looking into Sphinx and I do think we'll make the switch to sphinx for documentation. Just make sure it passes the DCO and we'll merge it in

@mvcrouse mvcrouse self-requested a review January 3, 2025 19:11
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants