Skip to content

Commit fedcc0f

Browse files
fix(workflow): use pull_request_target for plot-review
- Runs workflow from base branch (plot/*) not PR head - Ensures latest workflow version is used - Checkout PR head sha for actual code review
1 parent 3210393 commit fedcc0f

File tree

1 file changed

+6
-2
lines changed

1 file changed

+6
-2
lines changed

.github/workflows/plot-review.yml

Lines changed: 6 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -5,7 +5,9 @@ run-name: "Review: ${{ github.event.pull_request.head.ref }}"
55
# Reviews the implementation quality using AI
66

77
on:
8-
pull_request:
8+
# Use pull_request_target to run from base branch (has secrets access)
9+
# This ensures we use the latest workflow version from main/plot/* branch
10+
pull_request_target:
911
types: [opened, synchronize]
1012
branches:
1113
- 'plot/**' # PRs targeting feature branches
@@ -39,9 +41,11 @@ jobs:
3941
4042
echo "::notice::PR #$PR_NUMBER for $LIBRARY implementation of $SPEC_ID"
4143
42-
- name: Checkout code
44+
- name: Checkout PR code
4345
uses: actions/checkout@v6
4446
with:
47+
# Checkout PR head to review the actual implementation
48+
ref: ${{ github.event.pull_request.head.sha }}
4549
fetch-depth: 0
4650

4751
- name: Get main issue number from PR

0 commit comments

Comments
 (0)