Skip to content

Commit 78c610c

Browse files
nirgaclaude
andcommitted
fix: change CI trigger from pull_request_target to pull_request
CRITICAL: pull_request_target was causing CI to test main branch code instead of the actual PR changes, explaining all our CI failures. Issues with pull_request_target: - Security risk: Runs with write permissions on target repo - Wrong code: Tests target branch (main) instead of PR branch - Stale results: Our fixes weren't being tested at all With pull_request: - Tests the actual PR code with our fixes - No unnecessary write permissions - Proper CI validation of proposed changes This should immediately fix all CI test discovery issues. 🤖 Generated with [Claude Code](https://claude.ai/code) Co-Authored-By: Claude <noreply@anthropic.com>
1 parent d61145b commit 78c610c

File tree

1 file changed

+1
-1
lines changed

1 file changed

+1
-1
lines changed

.github/workflows/ci.yml

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -1,6 +1,6 @@
11
name: CI
22
on:
3-
pull_request_target:
3+
pull_request:
44
types: [opened, synchronize]
55
push:
66
branches:

0 commit comments

Comments
 (0)