Skip to content

Commit e621a7f

Browse files
ci(audience-sdk): scope path filters to source-affecting paths only (SDK-268)
Replaces the blanket src/Packages/Audience/** and examples/audience/** match with explicit allow-lists, so doc-only PRs (README.md, gitignore, etc.) no longer schedule the 12-cell PlayMode matrix or the .NET unit tests. Triggers now require a change to one of: - src/Packages/Audience/Runtime/** - src/Packages/Audience/Tests/** - src/Packages/Audience/package.json - src/Packages/Audience/Directory.Build.props - src/Packages/Audience/link.xml (IL2CPP stripping config) - examples/audience/Assets/** (sample-app workflow only) - examples/audience/Packages/** (sample-app workflow only) - examples/audience/ProjectSettings/** (sample-app workflow only) - the workflow file itself paths-ignore is not viable here: it only suppresses a workflow when every changed file matches the ignore list, so a mixed PR (markdown + package.json) would still trigger. Allow-listing is the only filter shape that handles this correctly. package.json stays in the trigger because real version, dependency, and unity-minimum changes live there and the matrix is exactly what should catch a typo. The cost of running on a metadata-only edit is the price of treating package.json as load-bearing. link.xml is included because it controls which managed types survive IL2CPP stripping in Player builds. A change there can produce runtime-only failures that only the PlayMode matrix can catch. Linear: SDK-268 Co-Authored-By: Claude Opus 4.7 (1M context) <noreply@anthropic.com>
1 parent 41ffc5b commit e621a7f

2 files changed

Lines changed: 18 additions & 4 deletions

File tree

.github/workflows/test-audience-sample-app.yml

Lines changed: 8 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -3,8 +3,14 @@ name: Audience SDK — PlayMode (IL2CPP + Mono)
33
on:
44
pull_request:
55
paths:
6-
- 'src/Packages/Audience/**'
7-
- 'examples/audience/**'
6+
- 'src/Packages/Audience/Runtime/**'
7+
- 'src/Packages/Audience/Tests/**'
8+
- 'src/Packages/Audience/package.json'
9+
- 'src/Packages/Audience/Directory.Build.props'
10+
- 'src/Packages/Audience/link.xml'
11+
- 'examples/audience/Assets/**'
12+
- 'examples/audience/Packages/**'
13+
- 'examples/audience/ProjectSettings/**'
814
- '.github/workflows/test-audience-sample-app.yml'
915
workflow_dispatch:
1016

.github/workflows/test-audience-sdk.yml

Lines changed: 10 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -4,11 +4,19 @@ on:
44
push:
55
branches: [main]
66
paths:
7-
- 'src/Packages/Audience/**'
7+
- 'src/Packages/Audience/Runtime/**'
8+
- 'src/Packages/Audience/Tests/**'
9+
- 'src/Packages/Audience/package.json'
10+
- 'src/Packages/Audience/Directory.Build.props'
11+
- 'src/Packages/Audience/link.xml'
812
- '.github/workflows/test-audience-sdk.yml'
913
pull_request:
1014
paths:
11-
- 'src/Packages/Audience/**'
15+
- 'src/Packages/Audience/Runtime/**'
16+
- 'src/Packages/Audience/Tests/**'
17+
- 'src/Packages/Audience/package.json'
18+
- 'src/Packages/Audience/Directory.Build.props'
19+
- 'src/Packages/Audience/link.xml'
1220
- '.github/workflows/test-audience-sdk.yml'
1321

1422
jobs:

0 commit comments

Comments
 (0)