Skip to content

Commit ae3498c

Browse files
dsarnoclaude
andauthored
ci(unity-tests): include MCPForUnity/Runtime/** in trigger paths (#1108)
Both the push and pull_request_target paths filters listed MCPForUnity/Editor/** but not Runtime/, so a PR (or push) that modified only files under MCPForUnity/Runtime/** would not trigger Unity tests at all -- including the safe-to-test label flow on a fork PR. CodeRabbit flagged this as a Low/💤 nitpick on #1103; PR #1106 made it a concrete recurrence: applying safe-to-test had no effect because the Runtime-only diff was filtered out. Adding Runtime/** mirrors the asmdef layout (Editor and Runtime are the two assembly roots whose code can break Unity compilation), matches the codebase's "domain symmetry" convention, and closes the silent-skip path the label gate was designed to prevent. Co-authored-by: Claude Opus 4.7 (1M context) <noreply@anthropic.com>
1 parent 8942e8b commit ae3498c

1 file changed

Lines changed: 2 additions & 0 deletions

File tree

.github/workflows/unity-tests.yml

Lines changed: 2 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -14,6 +14,7 @@ on:
1414
paths:
1515
- TestProjects/UnityMCPTests/**
1616
- MCPForUnity/Editor/**
17+
- MCPForUnity/Runtime/**
1718
- .github/workflows/unity-tests.yml
1819
# Fork PRs: maintainer applies the 'safe-to-test' label after reviewing
1920
# the diff. The workflow runs with UNITY_LICENSE in scope against the
@@ -25,6 +26,7 @@ on:
2526
paths:
2627
- TestProjects/UnityMCPTests/**
2728
- MCPForUnity/Editor/**
29+
- MCPForUnity/Runtime/**
2830
- .github/workflows/unity-tests.yml
2931

3032
jobs:

0 commit comments

Comments
 (0)