adutra commented on code in PR #4453: URL: https://github.com/apache/polaris/pull/4453#discussion_r3259884879
########## CONTRIBUTING.md: ########## @@ -126,7 +126,13 @@ Contributors may use a variety of tools when preparing changes to Polaris, inclu * Regardless of how a change is produced, the individual submitting the pull request is considered the author of the contribution and is fully responsible for it. * The pull request author **must understand the implementation end-to-end** and be able to **explain and justify the design and code** during review. * Tools, including AI systems, **are not** considered contributors. **Responsibility and authorship remain with the human** submitting the change. + Do not list AI tools as authors or co-authors, and do not include model names, + model versions, prompts, or reasoning settings unless a maintainer explicitly asks + for them during review. * Contributors are encouraged to **disclose** significant AI assistance in the pull request description for transparency. + A suitable disclosure is: + > AI assistance was used to prepare this change. I have reviewed and understand Review Comment: True, but your point is valid for all the checkbox items that we have currently in the checklist. I'm playing devil's advocate here, btw; I'm not a huge fan of that checklist, but since we have it, this item imho looks like a good candidate for a new checkbox in that list. -- 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]
