-
Notifications
You must be signed in to change notification settings - Fork 3.3k
Add AI tooling info to pull request template #10850
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
base: trunk
Are you sure you want to change the base?
Conversation
|
The following accounts have interacted with this PR and/or linked issues. I will continue to update these lists as activity occurs. You can also manually ask me to refresh this list by adding the Core Committers: Use this line as a base for the props when committing in SVN: To understand the WordPress project's expectations around crediting contributors, please review the Contributor Attribution page in the Core Handbook. |
|
I'm not familiar with the conversation and Chesterton fences that went into IMO much better DX choice than expecting contributors - even if/after reading the guidelines- to intuitively know and remember what they need to proactively disclose / how granularly to go, etc. Also has the added benefit of guiding users to potential constructive uses of AI without being overbearing. |
|
@justlevine I think that's a good suggestion, but I'm wary of adding too much to the template since it is currently pretty streamlined. Maybe start with simple freeform and links to the (in progress) guidelines and then iterate once the guidelines are more mature? Maybe the guidelines could have such a list of bullet points incorporated into it for copying into the PR template. It could even have an example PR template as part of the guidelines. My primary concern is to get something out the door sooner than later to help address the influx of PRs, many of which appear to be authored with in AI. |
|
@westonruter yup agree 100%, especially the part that anything at all is better than nothing.
My hope is to create less cognitive load, so while I love the idea of providing some examples of what disclosure looks like, I think a copy->paste flow from an external ref document isn't worth the friction. Still, even if it's just 1 extra sentence in the (Related: Some fun reading at where folks are discussing the issues and potential tools GitHub could offer maintainers to triage slop https://github.com/orgs/community/discussions/185387) |
Would you add a suggestion for what you have in mind? |
|
|
||
| <!-- | ||
| You are free to use artificial intelligence (AI) tooling to contribute, but you must you disclose what tooling you are using and to what extent a pull request has been authored by AI. It is your responsibility to review and take responsibility for what AI generates. See the WordPress AI Guidelines: <https://make.wordpress.org/ai/handbook/ai-guidelines/>. | ||
| --> | ||
|
|
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Very much spit-balling and not married to any particular phrasing, just trying to list a plurality of acceptable usage as concisely as I can think to.
| <!-- | |
| You are free to use artificial intelligence (AI) tooling to contribute, but you must you disclose what tooling you are using and to what extent a pull request has been authored by AI. It is your responsibility to review and take responsibility for what AI generates. See the WordPress AI Guidelines: <https://make.wordpress.org/ai/handbook/ai-guidelines/>. | |
| --> | |
| <!-- | |
| You may AI tooling to contribute, but it is your responsibly to review, understand, and take ownership of everything the LLM generates. You MUST disclose what tooling was used and where it was used. | |
| For example, please specify if IDE Autocompletions used for code or docs; whether an agent or code harness provided the spec, created some methods or even the initial draft PR, backfilled tests, or generated any non-code assets; etc. See the WordPress AI Guidelines: <https://make.wordpress.org/ai/handbook/ai-guidelines/>. | |
| --> | |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Is enumerating all of the acceptable usage needed? Wouldn't that be better to leave for the contributor to read from the AI Guidelines as they evolve?
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
So that's my argument is that it's precisely because AI Guidelines (both ours and general best practices) still have some evolution to do that we need some sort of tangible direction to make it clear we don't need -or want- a play-by-play, but rather something that indicates the signal-to-noise ratio for reviewers.
And to be clear, that's nowhere a full enumeration of acceptable usage, but I do think the rest of what matters can be mostly inferred from those while remaining relatively evergreen ;-) . It's a lot easier to broadly/blindly chunk with a checklist, so I'm doing my best with the constraint and I'm happy to dedupe any you feel are redundant.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Should be You may use AI tooling to contribute. Missing use
Trac ticket: https://core.trac.wordpress.org/ticket/64587
This incorporates aspects of the PR template from the WordPress/performance repo, namely the new “Use of AI Tools” section.
This Pull Request is for code review only. Please keep all other discussion in the Trac ticket. Do not merge this Pull Request. See GitHub Pull Requests for Code Review in the Core Handbook for more details.