Skip to content

docs(ai_policy): add an AI attribution policy to CONTRIBUTING.rst#3406

Open
JohnVillalovos wants to merge 1 commit into
mainfrom
jlvillal/ai_attribution
Open

docs(ai_policy): add an AI attribution policy to CONTRIBUTING.rst#3406
JohnVillalovos wants to merge 1 commit into
mainfrom
jlvillal/ai_attribution

Conversation

@JohnVillalovos

Copy link
Copy Markdown
Member

Require AI generated contributions to indicate so in commit messages.

Copilot AI review requested due to automatic review settings June 10, 2026 04:20

Copilot AI left a comment

Copy link
Copy Markdown

Choose a reason for hiding this comment

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

Pull request overview

This PR updates the contributor guidelines to require attribution in commit messages when an AI coding assistant meaningfully contributed to a commit, by adding an Assisted-by: trailer convention.

Changes:

  • Add an “AI Attribution” section describing when/how to include an Assisted-by: commit trailer.
  • Provide an example commit message showing Closes: alongside Assisted-by:.

💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.

Comment thread CONTRIBUTING.rst Outdated
Comment thread CONTRIBUTING.rst Outdated
Comment thread CONTRIBUTING.rst
@JohnVillalovos JohnVillalovos force-pushed the jlvillal/ai_attribution branch 2 times, most recently from bf0e5f0 to f2ad992 Compare June 10, 2026 04:25
@JohnVillalovos JohnVillalovos requested review from Copilot and nejch June 10, 2026 04:26

Copilot AI left a comment

Copy link
Copy Markdown

Choose a reason for hiding this comment

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

Pull request overview

Copilot reviewed 1 out of 1 changed files in this pull request and generated no new comments.

@JohnVillalovos JohnVillalovos force-pushed the jlvillal/ai_attribution branch 3 times, most recently from aa92932 to 8106ca0 Compare June 10, 2026 04:41
@JohnVillalovos JohnVillalovos requested a review from Copilot June 10, 2026 04:41

Copilot AI left a comment

Copy link
Copy Markdown

Choose a reason for hiding this comment

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

Pull request overview

Copilot reviewed 2 out of 2 changed files in this pull request and generated 2 comments.

Comment thread docs/index.rst
Comment thread CONTRIBUTING.rst Outdated
@JohnVillalovos JohnVillalovos force-pushed the jlvillal/ai_attribution branch 4 times, most recently from a8ae5d6 to 40ddaba Compare June 10, 2026 04:56
Require AI generated contributions to indicate so in commit messages.

Also add the CONTRIBUTING.rst file to the generated documentation.
@JohnVillalovos JohnVillalovos force-pushed the jlvillal/ai_attribution branch from 40ddaba to 66007d3 Compare June 10, 2026 04:59
@codecov

codecov Bot commented Jun 10, 2026

Copy link
Copy Markdown

Codecov Report

✅ All modified and coverable lines are covered by tests.
✅ Project coverage is 95.78%. Comparing base (a4cbecb) to head (66007d3).

Additional details and impacted files
@@            Coverage Diff             @@
##             main    #3406      +/-   ##
==========================================
+ Coverage   92.23%   95.78%   +3.54%     
==========================================
  Files         100      100              
  Lines        6172     6172              
==========================================
+ Hits         5693     5912     +219     
+ Misses        479      260     -219     
Flag Coverage Δ
api_func_v4 83.86% <ø> (?)
cli_func_v4 78.74% <ø> (ø)
unit 90.32% <ø> (ø)

Flags with carried forward coverage won't be shown. Click here to find out more.
see 23 files with indirect coverage changes

🚀 New features to boost your workflow:
  • ❄️ Test Analytics: Detect flaky tests, report on failures, and find test suite problems.

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

@JohnVillalovos Nice, now you just need to add it to this commit as well ;-)

@JohnVillalovos

Copy link
Copy Markdown
Member Author

@JohnVillalovos Nice, now you just need to add it to this commit as well ;-)

If you look at all my failures along the way it kind of gives away that I didn't use AI this time 😄

@JohnVillalovos

Copy link
Copy Markdown
Member Author

But we do use this policy over at LibreBooking https://github.com/LibreBooking/librebooking and I also do use AI quite often to help. If I had been smarter I would have used AI but I was lazy and thought it can't be that hard. My knowledge of RST was not as good as I thought it was. I should have just started up Claude...

@max-wittig

Copy link
Copy Markdown
Member

I'm not a big fan of this. All code is now written by LLMs and humans are just nudging it in the right direction. I would have agreed with adding this in 2025, but our work fundamentally changed in 2026 so I'm not sure this makes sense.

The model should not matter, but the quality of the code does.

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

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants