raulcd commented on issue #47899: URL: https://github.com/apache/arrow/issues/47899#issuecomment-3430839785
I think checklists are nice both as a submitter and as a reviewer and I think they are friendlier to new contributors. It's easier to check a button than understanding what do I have to put on a free-form text. I agree with @zanmato1984 comments above that some of the checks on these checklist are similarly covered today either by our PR description or the linked issue. For example the **Are these changes tested?** section on our current PR description cover a similar case than the testing part presented on the checklist. Maybe we could still have part of the PR Description as is (rationale, changes included) but add a checklist section for `Are these changes tested?` and `Are there any user-facing changes?` -- 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]
