Skip to content

[nfc] update CONTRIBUTING.md to add basic LLM policy#22591

Open
silverweed wants to merge 1 commit into
root-project:masterfrom
silverweed:contributing
Open

[nfc] update CONTRIBUTING.md to add basic LLM policy#22591
silverweed wants to merge 1 commit into
root-project:masterfrom
silverweed:contributing

Conversation

@silverweed

Copy link
Copy Markdown
Contributor

This Pull request:

Adds a section about LLM usage to CONTRIBUTING.md. It is not meant as a fully-fledged "AI policy" but just some basic common-sense guidelines that are nonetheless good to spell out.

Also adds a section about branch names, since opening a PR with a branch named master conflicts with our CI.

@silverweed silverweed self-assigned this Jun 12, 2026
@silverweed silverweed requested a review from dpiparo as a code owner June 12, 2026 10:01

@ferdymercury ferdymercury left a comment

Copy link
Copy Markdown
Collaborator

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Thanks!
Maybe also mention that copy pasting could lead to copyright infringement eg if LLM is copying verbatim from a GPL project into LGPL

@github-actions

github-actions Bot commented Jun 12, 2026

Copy link
Copy Markdown

Test Results

0 tests   0 ✅  0s ⏱️
0 suites  0 💤
0 files    0 ❌

Results for commit 3771838.

♻️ This comment has been updated with latest results.

@silverweed silverweed added the skip ci Skip the full builds on the actions runners label Jun 12, 2026

@guitargeek guitargeek left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Very nice, I think this is well-written and captures the essential etiquette!

Going further, we might spell out precisely what it means "to disclose whether the proposed changes (code or otherwise) were automatically generated", maybe with templates for such disclosures that can be added to commit messages and PR descriptions.

@dpiparo

dpiparo commented Jun 15, 2026

Copy link
Copy Markdown
Member

Thanks for this initiative. Before merging, even if I do not see huge hurdles ahead, I was thinking to discuss this a bit broadly within the core developers.

@vepadulano vepadulano left a comment

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Thanks! A couple of minor suggestions from my side

Comment thread CONTRIBUTING.md
Please ping people :wave: should you not get timely feedback, for instance with `@root-project/core ping!`

> [!IMPORTANT]
> For compatibility with our CI, your should create your PR with a **branch name different from `master`**.

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Suggested change
> For compatibility with our CI, your should create your PR with a **branch name different from `master`**.
> For compatibility with our CI, you should create your PR with a **branch name different from `master`**.

Comment thread CONTRIBUTING.md

Thank you for reading this; and even more: thank you :bouquet: for considering to contribute!

## LLM Disclosure

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

I don't know if this is a term already being used elsewhere, but to be more general we could think about calling it "AI-assisted coding disclosure".

Comment thread CONTRIBUTING.md
Please avoid proposing LLM-generated changes as-is without a full review and understanding, as this disrespects
the reviewers' time and may cause your future contributions to be automatically rejected if reiterated.

Likewise, PR comments and any sort of engagement with other contributors must not be parroting LLMs or chatbots.

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Is "parroting" a universally understood term?

Suggested change
Likewise, PR comments and any sort of engagement with other contributors must not be parroting LLMs or chatbots.
Likewise, PR comments and any sort of engagement with other contributors must not be a simple copy of an LLM's or chatbot's output.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

skip ci Skip the full builds on the actions runners

Projects

None yet

Development

Successfully merging this pull request may close these issues.

7 participants