Skip to content

Fix Copilot review workflow to trigger on fork PRs#1074

Merged
liamfallon merged 1 commit into
kptdev:mainfrom
Nordix:fix_copilot_action
Jun 25, 2026
Merged

Fix Copilot review workflow to trigger on fork PRs#1074
liamfallon merged 1 commit into
kptdev:mainfrom
Nordix:fix_copilot_action

Conversation

@efiacor

@efiacor efiacor commented Jun 24, 2026

Copy link
Copy Markdown
Contributor

Title

Fix Copilot review workflow to trigger on fork PRs


Description

  • What changed: Changed on: pull_request to on: pull_request_target in the Copilot review workflow
  • Why it's needed: pull_request doesn't trigger for PRs from forks (no write permissions). pull_request_target runs in the base repo context with the necessary pull-requests: write permission to request reviewers.

Related Issue(s)

  • N/A

Type of Change

  • Bug fix
  • New feature
  • Enhancement
  • Refactor
  • Documentation
  • Tests
  • Other: ________

Checklist

  • Code follows project style guidelines
  • Self-reviewed changes
  • Tests added/updated
  • Documentation added/updated
  • All tests and gating checks pass

Testing Instructions (Optional)

  1. Open a PR from a fork
  2. Verify Copilot is requested as a reviewer

Additional Notes (Optional)

  • N/A

AI Disclosure

  • I have used AI in the creation of this PR.

If so, please describe how:

  • The author has fully verified all code.

Signed-off-by: Fiachra Corcoran <fiachra.corcoran@est.tech>
@efiacor efiacor self-assigned this Jun 24, 2026
@efiacor efiacor requested review from a team June 24, 2026 14:18
@dosubot dosubot Bot added the size:XS This PR changes 0-9 lines, ignoring generated files. label Jun 24, 2026
@netlify

netlify Bot commented Jun 24, 2026

Copy link
Copy Markdown

Deploy Preview for kpt-porch ready!

Name Link
🔨 Latest commit 88da935
🔍 Latest deploy log https://app.netlify.com/projects/kpt-porch/deploys/6a3be7268e4e0f000805a97e
😎 Deploy Preview https://deploy-preview-1074--kpt-porch.netlify.app
📱 Preview on mobile
Toggle QR Code...

QR Code

Use your smartphone camera to open QR code link.
🤖 Make changes Run an agent on this branch

To edit notification comments on pull requests, go to your Netlify project configuration.

@sonarqubecloud

Copy link
Copy Markdown

@dosubot dosubot Bot added the lgtm #ededed label Jun 25, 2026
@liamfallon liamfallon merged commit cb1ba1c into kptdev:main Jun 25, 2026
30 of 32 checks passed
@efiacor efiacor deleted the fix_copilot_action branch June 25, 2026 08:34
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

lgtm #ededed size:XS This PR changes 0-9 lines, ignoring generated files.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants