Skip to content

fix(code-review): add "Copilot" user login special case to bot check#107861

Merged
srest2021 merged 3 commits intomasterfrom
srest2021/CW-791
Feb 10, 2026
Merged

fix(code-review): add "Copilot" user login special case to bot check#107861
srest2021 merged 3 commits intomasterfrom
srest2021/CW-791

Conversation

@srest2021
Copy link
Member

@srest2021 srest2021 commented Feb 9, 2026

Fixes CW-791

Per the eligibility check logs (example) Copilot uses the "Copilot" login without the "[bot]" suffix so let's add it to the bot check.

@linear
Copy link

linear bot commented Feb 9, 2026

@github-actions github-actions bot added the Scope: Backend Automatically applied to PRs that change backend components label Feb 9, 2026
@srest2021 srest2021 marked this pull request as ready for review February 9, 2026 18:04
Copy link
Contributor

@cursor cursor bot left a comment

Choose a reason for hiding this comment

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

Cursor Bugbot has reviewed your changes and found 1 potential issue.

Bugbot Autofix is OFF. To automatically fix reported issues with Cloud Agents, enable Autofix in the Cursor dashboard.

@srest2021 srest2021 requested a review from a team February 9, 2026 18:28
Copy link
Member

@billyvg billyvg left a comment

Choose a reason for hiding this comment

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

alias is the username right, so it should be unique

@srest2021
Copy link
Member Author

@billyvg yes alias corresponds to user login

"alias": user["login"],

@srest2021 srest2021 merged commit 6792444 into master Feb 10, 2026
72 checks passed
@srest2021 srest2021 deleted the srest2021/CW-791 branch February 10, 2026 17:32
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

Scope: Backend Automatically applied to PRs that change backend components

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants