Skip to content

SPO-912: Fix copilot reviewer default and rename job ID in react-ditto security workflow#88

Merged
John Cunningham (SausCode) merged 2 commits intomasterfrom
johncunningham/spo-912-fix-copilot-reviewer-default-and-rename-job-id-in-react
May 7, 2026
Merged

SPO-912: Fix copilot reviewer default and rename job ID in react-ditto security workflow#88
John Cunningham (SausCode) merged 2 commits intomasterfrom
johncunningham/spo-912-fix-copilot-reviewer-default-and-rename-job-id-in-react

Conversation

@SausCode
Copy link
Copy Markdown
Contributor

SPO-912: Fix copilot reviewer default and rename job ID in react-ditto security workflow

SPO-912

John Cunningham (SausCode) and others added 2 commits May 7, 2026 12:21
Remove copilot from the reviewers input default. @copilot is already
hardcoded in the reusable workflow in sec-tools-public; having it in
the caller default caused a duplicate request without the @ prefix,
which fails with "Could not resolve user with login 'copilot'".

Closes: SPO-912

Co-Authored-By: Claude Opus 4.6 (1M context) <noreply@anthropic.com>
@SausCode John Cunningham (SausCode) marked this pull request as ready for review May 7, 2026 19:22
Copy link
Copy Markdown

@kk-0110 Kevin Killoran (kk-0110) left a comment

Choose a reason for hiding this comment

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

lgtm

@SausCode John Cunningham (SausCode) merged commit 0d98051 into master May 7, 2026
1 check passed
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.

2 participants