Skip to content

Fix GitHub Copilot CLA allowlist to prevent false CLA requirement - #31

Merged
m7pr merged 3 commits into
mainfrom
copilot/fix-30
Aug 22, 2025
Merged

m7pr merged 3 commits into
mainfrom
copilot/fix-30

Conversation

Copilot AI commented Aug 22, 2025 •

Copy link
Copy Markdown
Contributor

This PR fixes an issue where GitHub Copilot was incorrectly being required to sign the CLA despite being added to the allowlist.

Problem

The CLA workflow had copilot in the allowlist, but GitHub Copilot appears with different username formats when making contributions (such as github-copilot[bot] or copilot[bot]). This caused the CLA Assistant to incorrectly flag Copilot contributions as requiring CLA signature, as shown in the issue screenshot where @copilot was marked with a red X and required to sign the CLA.

Solution

Updated the CLA workflow allowlist to include the correct GitHub Copilot bot formats:

  • Added github-copilot[bot] (most common GitHub Copilot format)
  • Added copilot[bot] (alternative GitHub Copilot format)
  • Kept existing copilot entry for backward compatibility
  • Reformatted the allowlist as multi-line YAML for better readability

The allowlist now properly covers all possible GitHub Copilot username variations while maintaining compatibility with existing entries.

Changes

# Before
allowlist: github-actions[bot],dependabot-preview[bot],insights-engineering-bot,dependabot[bot],copilot

# After  
allowlist: >-
  github-actions[bot],
  dependabot-preview[bot],
  insights-engineering-bot,
  dependabot[bot],
  copilot,
  github-copilot[bot],
  copilot[bot]

Fixes #30.


💡 You can make Copilot smarter by setting up custom instructions, customizing its development environment and configuring Model Context Protocol (MCP) servers. Learn more Copilot coding agent tips in the docs.

Copilot AI commented Aug 22, 2025

Copy link
Copy Markdown
Contributor Author

@m7pr 👋 This repository doesn't have Copilot instructions. With Copilot instructions, I can understand the repository better, work faster and produce higher quality PRs.

I can generate a .github/copilot-instructions.md file for you automatically. Click here to open a pre-filled issue and assign it to me. I'll write the instructions, and then tag you for review.

Co-authored-by: m7pr <133694481+m7pr@users.noreply.github.com>
@m7pr m7pr added the core label Aug 22, 2025
@m7pr
m7pr requested a review from walkowif August 22, 2025 08:28
@github-actions

github-actions Bot commented Aug 22, 2025 •

Copy link
Copy Markdown


🎉 Thank you for your contribution! Before this PR can be accepted, we require that you all read and agree to our Contributor License Agreement.
You can digitally sign the CLA by posting a comment on this Pull Request in the format shown below. This agreement will apply to this PR as well as all future contributions on this repository.


I have read the CLA Document and I hereby sign the CLA


1 out of 2 committers have signed the CLA.
✅ (m7pr)[https://github.com/m7pr]
❌ @Copilot
You can retrigger this bot by commenting recheck in this Pull Request. Posted by the CLA Assistant Lite bot.

@m7pr
m7pr marked this pull request as ready for review August 22, 2025 08:29
Copilot AI changed the title [WIP] [Feature Request]: Remove Copilot form allowlist in CLA test Fix GitHub Copilot CLA allowlist to prevent false CLA requirement Aug 22, 2025
Copilot AI requested a review from m7pr August 22, 2025 08:30
Signed-off-by: Marcin <133694481+m7pr@users.noreply.github.com>
@m7pr

m7pr commented Aug 22, 2025

Copy link
Copy Markdown
Contributor

I have read the CLA Document and I hereby sign the CLA

@m7pr
m7pr merged commit 9d853a3 into main Aug 22, 2025
1 check failed
@m7pr
m7pr deleted the copilot/fix-30 branch August 22, 2025 08:34
@github-actions github-actions Bot locked and limited conversation to collaborators Aug 22, 2025
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

Labels

Projects

None yet

Development

Successfully merging this pull request may close these issues.

[Feature Request]: Remove Copilot form allowlist in CLA test

3 participants