Skip to content

feat: Narrow down allowlist in CLA#138

Merged
alessfg merged 1 commit intomainfrom
update-cla
Mar 4, 2026
Merged

feat: Narrow down allowlist in CLA#138
alessfg merged 1 commit intomainfrom
update-cla

Conversation

@alessfg
Copy link
Member

@alessfg alessfg commented Mar 4, 2026

Proposed changes

Allowing all bots to skip the CLA is too permissive and does not work for tooling such as Copilot.

Checklist

Before creating a PR, run through this checklist and mark each as complete:

Co-Authored-By: Alessandro Fael Garcia <alessfg@hotmail.com>
Copilot AI review requested due to automatic review settings March 4, 2026 10:13
@alessfg alessfg requested a review from a team March 4, 2026 10:14
@alessfg alessfg self-assigned this Mar 4, 2026
@alessfg alessfg added the enhancement New feature or request label Mar 4, 2026
Copy link

Copilot AI left a comment

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 tightens the CLA assistant configuration by replacing a broad bot wildcard allowlist with an explicit set of permitted bot accounts, reducing the chance of unintended CLA bypasses.

Changes:

  • Replaced the bot* CLA allowlist wildcard with an explicit list of allowed bot usernames.

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

Copy link
Member

@buulam buulam left a comment

Choose a reason for hiding this comment

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

Approved

@alessfg alessfg merged commit 87d2a62 into main Mar 4, 2026
6 checks passed
@alessfg alessfg deleted the update-cla branch March 4, 2026 13:21
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

enhancement New feature or request

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants