Fokko opened a new issue, #7776: URL: https://github.com/apache/iceberg/issues/7776
### Feature Request / Improvement Since we're [releasing docstrings](https://github.com/apache/iceberg/pull/7108), thanks @LuigiCerone! 🙌🏻 We should make sure that we keep our docstrings tidy. Currently, there are quite a few violations: - D100: Missing docstring in public module - D101: Missing docstring in public class - D102: Missing docstring in public method - D103: Missing docstring in public function - D104: Missing docstring in public package - D105: Missing docstring in magic method - D106: Missing docstring in public nested class - D107: Missing docstring in __init__ - D200: One-line docstring should fit on one line with quotes - D202: No blank lines allowed after function docstring - D205: 1 blank line required between summary line and description - D209: Multi-line docstring closing quotes should be on a separate line - D400: First line should end with a period - D401: First line should be in imperative mood - D403: First word of the first line should be properly capitalized - D414: Section has no content ('Returns') To fix this, we need to remove the violation code from `.pre-commit-config.yaml`, run pre-commit `make lint`, fix the violations, and profit! ### Query engine None -- This is an automated message from the Apache Git Service. To respond to the message, please log on to GitHub and use the URL above to go to the specific comment. To unsubscribe, e-mail: [email protected] For queries about this service, please contact Infrastructure at: [email protected] --------------------------------------------------------------------- To unsubscribe, e-mail: [email protected] For additional commands, e-mail: [email protected]
