Adding AI disclosures updates#259
Conversation
jbonofre
left a comment
jbonofre
left a comment
There was a problem hiding this comment.
First glance: I love it 😄
I will do the review/merge tomorrow morning my time.
| (e.g., `Co-authored-by: Copilot <copilot@github.com>` or a note in the PR) | ||
|
|
||
| This is not a gate — it is a transparency practice that helps the community | ||
| understand how contributions are produced. |
There was a problem hiding this comment.
Wondering if this is required by Apache ? Else I think we should skip it since directionally it looks like most PRs would be assisted by AI tools.
cc @jbonofre
We also want to keep our PR template bounded.
There was a problem hiding this comment.
This is not strictly required from ASF standpoint. Especially the co-authored-by statement is not mandatory.
At the ASF, we might ask the AI tool used mostly to verify there's no legal issue (trademark, IP, etc).
I'm expecting from this PR just a link to https://www.apache.org/legal/generative-tooling.html for instance.
There was a problem hiding this comment.
The Apache-2.0 license, and the Apache Individual Contribution License Agreement, both remind contributors that they are responsible for disclosing any copyrighted materials in submitted contributions that are not their original creation. This is as true when using generative AI tooling, as it is when using materials from public websites or code from other open-source projects.
There was a problem hiding this comment.
I think the critical part is that each person is responsible for their contribution.
I think the "co-authored" is a little funny. I thought AI had no copyright claims on the code it generated. Also, it is common for me to use multiple harnesses (codex, claudecode, opencode, cursor) and multiple models with them (opus, sonnet, gpt, composer (and I expect some GLM in the future)).
Should we list all of them?
There was a problem hiding this comment.
O.k. I looked at the generative-tooling link. If you think the harness and/or model helps later analysis. We can add it. However, let's cover the "multi" case and whether we want models as well as harness.
5 participants
Summary
This PR adds section to CONTRIBUTING.md and adds optional disclosure checkbox as part of PR template.