Skip to content

Add a policy for AI code. - #652

Open
samcunliffe wants to merge 2 commits into
mainfrom
sc/ai-policy-649
Open

samcunliffe wants to merge 2 commits into
mainfrom
sc/ai-policy-649

Conversation

@samcunliffe

Copy link
Copy Markdown
Member

Referencing the ARC-wide AI dev policy.
Resolves #649.

Referencing the ARC-wide AI dev policy.
Resolves #649.

Copilot AI 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.

🟡 Changes recommended

The policy link is ambiguous and does not explicitly address agent-only pull requests.

2 open findings
What changed in this PR

Adds AI-assisted contribution guidance to CONTRIBUTING.md, referencing the ARC-wide policy.

Changes:

  • Documents acceptable AI assistance and contributor responsibility.
  • Adds ARC policy references.
  • Clarifies expectations for focused pull requests.
File Description
CONTRIBUTING.md Adds AI contribution guidance and policy references.

🧠 Review effort: Lite


💡 Add a code-review agent skill or configure MCP servers for context-aware, tailored reviews. Learn more in the docs.

Comment thread CONTRIBUTING.md
Comment thread CONTRIBUTING.md Outdated
Co-authored-by: Copilot Autofix powered by AI <175728472+Copilot@users.noreply.github.com>
@samcunliffe samcunliffe self-assigned this Oct 12, 2026
@samcunliffe samcunliffe added documentation Improvements or additions to documentation task A task to be completed with the repo needs-2-reviewers Could be considered "controversial" so worth a second pair of eyes labels Oct 12, 2026
@samcunliffe
samcunliffe requested a review from a team October 12, 2026 16:21
@samcunliffe
samcunliffe marked this pull request as ready for review October 12, 2026 16:21

@paddyroddy paddyroddy 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.

Ace 👌

@samcunliffe
samcunliffe requested review from a team and matt-graham October 13, 2026 10:31

@Saransh-cpp Saransh-cpp 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.

Hi both :)

I looked at this policy when it was going around and it looks amazing! Do you also have something for people who are not "owners" / "maintainers" of a project? More specifically, new contributors to an open-source project (for instance, a UCL student sending a PR to this repository).

I am having quite a few discussions around this topic, and while I am completely against newcomers using AI (as that defeats the point of them learning things), there are a quite few valid points in favour of allowing it (they'll start hiding AI usage if we keep closing AI PRs, AI adoption is accelerating and we can't run away from it, some repositories have failed to pick up long-lasting contributors even before the AI era, ...).

I could not find anything similar in the policy doc, but given that ARC came up with the doc, I was wondering if there have been similar discussions within ARC?

Thanks a lot!

This branch has not been deployed

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

Labels

documentation Improvements or additions to documentation needs-2-reviewers Could be considered "controversial" so worth a second pair of eyes task A task to be completed with the repo

Projects

None yet

Development

Successfully merging this pull request may close these issues.

Should we have an AI policy?

4 participants