jbonofre commented on code in PR #4453: URL: https://github.com/apache/polaris/pull/4453#discussion_r3263568801
########## 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: A checkbox is a good idea. It will clearly indicate the contributor that he should disclose it. ########## 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. Review Comment: If I fully agree that AI tools should not be author/co-author, I think it's ok to share details about the model, tool, etc. It's up to the contributor. The minimum is to disclose he's using AI, he can share more if he wants. -- 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]
