Skip to content

ci: bump cla-github-action to 73f6929 (email-based allowlist)#89

Open
iainmcgin wants to merge 1 commit intomainfrom
iain/cla-action-bump
Open

ci: bump cla-github-action to 73f6929 (email-based allowlist)#89
iainmcgin wants to merge 1 commit intomainfrom
iain/cla-action-bump

Conversation

@iainmcgin
Copy link
Copy Markdown
Collaborator

Bumps iainmcgin/cla-github-action to 73f6929 and allowlists noreply@anthropic.com.

The new revision adds email-based matching to the allowlist alongside the existing name-based match. AI assistants commonly add Co-authored-by: trailers like Claude Opus 4.7 <noreply@anthropic.com>, and the impersonation guard added in the previous bump correctly counts those as committers — without an email allowlist, every PR with such a trailer fails the CLA check. Allowlisting noreply@anthropic.com suppresses the synthetic identity. The PR opener still has to sign.

Tests on the action itself: 100/100 pass, including new cases for exact email match, *@anthropic.com glob, and the no-email-supplied path.

Pre-emptive on this repo — connect-rust hit this in PR anthropics/connect-rust#63 (fix tracked in anthropics/connect-rust#74); buffa will encounter the same trailer pattern soon enough.

The new revision adds email-based matching to the allowlist alongside
the existing name-based match. AI assistants commonly add
Co-authored-by: trailers like 'Claude Opus 4.7 <noreply@anthropic.com>',
and the impersonation guard counts those as committers — without an
email allowlist, every PR with such a trailer fails the CLA check.
Allowlisting 'noreply@anthropic.com' suppresses the synthetic identity;
the PR opener still has to sign.
@github-actions
Copy link
Copy Markdown

All contributors have signed the CLA ✍️ ✅
Posted by the CLA Assistant Lite bot.

@iainmcgin iainmcgin marked this pull request as ready for review April 30, 2026 23:46
@iainmcgin iainmcgin requested a review from asacamano April 30, 2026 23:46
@iainmcgin iainmcgin enabled auto-merge (squash) April 30, 2026 23:48
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.

1 participant