Skip to content

best-practices: add guidance on AI-generated commits and AI-assisted PR workflows - #61

Merged
paddybyers merged 3 commits into
mainfrom
feature./update-best-practices-for-ai
Sep 24, 2026
Merged

paddybyers merged 3 commits into
mainfrom
feature./update-best-practices-for-ai

Conversation

@paddybyers

Copy link
Copy Markdown
Member

Summary

  • Add an "AI-generated commits" section to commits.md, establishing that AI-produced commits must meet the same standards for coherence, atomicity, and message quality, and that AI provenance should be indicated
  • Add a "Use of coding agents and AI review bots" section to pull-requests.md, clarifying expectations for AI-raised PRs, AI-assisted reviews (draft state), and author responsibility to personally review AI-generated code before requesting peer review

Test plan

  • Review that the new sections are consistent with the existing guidance in each document
  • Verify cross-references between the two documents are correct

🤖 Generated with Claude Code

Comment thread best-practices/pull-requests.md Outdated
Comment thread best-practices/commits.md Outdated

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

Good improvements. I've added some comments that need review.

Comment thread best-practices/commits.md Outdated
Comment thread best-practices/pull-requests.md
Comment thread best-practices/pull-requests.md Outdated
@mattheworiordan

Copy link
Copy Markdown
Member

This has been stagnant for 2-3 months now @paddybyers

paddybyers and others added 2 commits September 13, 2026 17:44
…PR workflows

Coding agents are now a significant part of how we produce code.
These additions establish clear expectations for their use within
our existing commit and PR standards.

Co-Authored-By: Claude Opus 4.6 <noreply@anthropic.com>
@paddybyers
paddybyers force-pushed the feature./update-best-practices-for-ai branch from 72ed073 to 46e13b2 Compare September 13, 2026 18:23
@paddybyers

Copy link
Copy Markdown
Member Author

I have updated; PTAL.

@mattheworiordan

Copy link
Copy Markdown
Member

Thanks, looks good.

How is this going to be used @paddybyers?

In addition to communicating with the team, Is this not also a shared context/skill/plugin that runs automatically to help people when creating PRs, effectively acting as your assistant to help people implement this? What is your thinking on how we use this guidance vs write it it down?

Comment thread best-practices/commits.md

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

Couple of minor comments, but LGTM

Comment thread best-practices/commits.md
Comment thread best-practices/pull-requests.md
@paddybyers
paddybyers merged commit 4c9a1f2 into main Sep 24, 2026
1 check passed
@paddybyers
paddybyers deleted the feature./update-best-practices-for-ai branch September 24, 2026 07:10
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Development

Successfully merging this pull request may close these issues.

6 participants