chore: Exempt metamaskbotv2 from CLA check cp-7.71.0#27758
Conversation
The CLABot workflow has been updated to exempt `metamaskv2` (i.e. commits created by Patroll tokens) from the CLA check. We saw the CLA check fail recently on a release branch due to some commits appearing for the first time from Patroll (see #27708 (comment)). This change will fix that CI failure.
| # This branch can't have protections, commits are made directly to the specified branch. | ||
| branch: 'cla-signatures' | ||
| allowlist: 'dependabot[bot],metamaskbot,crowdin-bot,runway-github[bot],cursorbot,cursoragent' | ||
| allowlist: 'dependabot[bot],metamaskbot,metamaskbotv2[bot],crowdin-bot,runway-github[bot],cursorbot,cursoragent' |
There was a problem hiding this comment.
We've needed metamaskbotv2[bot] on the allowlist on extension for a bit longer, see here for reference: https://github.com/MetaMask/metamask-extension/blob/c5247c5d3b45d85a99258ac3dbcdc2554615edfa/.github/workflows/cla.yml#L30
🔍 Smart E2E Test Selection
click to see 🤖 AI reasoning detailsE2E Test Selection:
This workflow only handles Contributor License Agreement checking for pull requests. It has absolutely no impact on:
No E2E tests need to run for this change. The CLA workflow is completely isolated from the app's test suite. Performance Test Selection: |
metamaskbotv2 from CLA checkmetamaskbotv2 from CLA check cp-7.71.0
There was a problem hiding this comment.
Cursor Bugbot has reviewed your changes and found 1 potential issue.
Bugbot Autofix is OFF. To automatically fix reported issues with cloud agents, have a team admin enable autofix in the Cursor dashboard.
|
Description
The CLABot workflow has been updated to exempt
metamaskv2(i.e. commits created by Patroll tokens) from the CLA check.We saw the CLA check fail recently on a release branch due to some commits appearing for the first time from Patroll (see #27708 (comment)). This change will fix that CI failure.
Changelog
CHANGELOG entry: null
Related issues
N/A
Manual testing steps
N/A
Screenshots/Recordings
N/A
Pre-merge author checklist
Pre-merge reviewer checklist
Note
Low Risk
Low risk workflow change limited to expanding the CLA bot allowlist; main impact is potentially skipping CLA enforcement for this additional bot account.
Overview
Updates the
CLA Signature BotGitHub Actions workflow to addmetamaskbotv2[bot]to the CLA exemption allowlist, preventing CLA check failures on PRs/merge groups created by that bot.Written by Cursor Bugbot for commit f1f4342. This will update automatically on new commits. Configure here.