Repository navigation
ci: reject commits authored by Claude or Copilot - #5869
Conversation
|
@dgryski @jakebailey @deadprogram despite AGENTS.md, co-attribution commits do appear in PRs. This wastes maintainer bandwidth and additional cleanup work if they're not caught before merging -- of which I have been the most recent offender. A job like this can automate the maintenance burden. In the future it could also be made a required build on the repo to prevent merges with co-attribution commits. |
e477b7a to
d4eaaaf
Compare
|
Thanks for adding this @davecheney. I ran the pattern on my local machine against a set of sample trailers and the full
What do you think of these? |
|
I think good enough is better than perfect here, it’ll Let me take another swing at making this wider, not tighter, with the view that this will probably be a perpetual work in progress |
deadprogram
left a comment
There was a problem hiding this comment.
Thanks for helping reduce the noise @davecheney
The earlier version also read commit messages and the PR description. That caused too much debate about which patterns to match, so this version starts small. It checks only the author and committer email of each commit. Co-authored-by trailers and PR text are not checked. We can add more checks in later PRs if we need them.