rodrigc commented on PR #1530: URL: https://github.com/apache/iceberg-python/pull/1530#issuecomment-2597332238
I see what you are saying. It looks like this issue has come up before: https://github.com/pre-commit/pre-commit-hooks/issues/159 This project chose to remove the check-docstring-first pre-commit hook: https://github.com/tiacsys/verylittlewire/commit/07b2cd225524308a2e158bafbc48b7e0fd712e49 Do you suggest that pyiceberg do the same? Should that sort of change be done in a separate PR, or can it be combined with this one? -- 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: issues-unsubscr...@iceberg.apache.org For queries about this service, please contact Infrastructure at: us...@infra.apache.org --------------------------------------------------------------------- To unsubscribe, e-mail: issues-unsubscr...@iceberg.apache.org For additional commands, e-mail: issues-h...@iceberg.apache.org
