flyingImer commented on code in PR #4453: URL: https://github.com/apache/polaris/pull/4453#discussion_r3270207184
########## 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: 2 cents: split into: - "AI tools are not authors or co-authors. Do not list them on commits, in commit messages (no `Co-Authored-By:` lines), in `Author:` lines, or in PR metadata." - "Contributors may disclose model, tool, or workflow details in the PR description. Maintainers may ask for more context during review." ########## 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 + > the full diff, and I take responsibility for the contribution. Review Comment: 2 cents: frame as one example. > Contributors may use a disclosure such as: "AI assistance was used to prepare this change. I have reviewed and understand the full diff, and I take responsibility for the contribution." The concrete sentence still helps newcomers and doesn't crowd out contributors who want to say more. -- 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]
