Skip to content

Commit 5c5fe74

Browse files
docdyhrclaude
andcommitted
fix(ci): wrap cross-platform-validation inputs in explicit expressions
Use explicit ${{ }} expression wrapping for job-level if-conditions that reference github.event.inputs.test_type. GitHub validates workflow files on push even when the workflow isn't triggered by push, and bare expressions referencing event inputs may fail validation. Co-Authored-By: Claude Opus 4.6 <noreply@anthropic.com>
1 parent 1ee6379 commit 5c5fe74

1 file changed

Lines changed: 3 additions & 3 deletions

File tree

.github/workflows/cross-platform-validation.yml

Lines changed: 3 additions & 3 deletions
Original file line numberDiff line numberDiff line change
@@ -104,7 +104,7 @@ jobs:
104104
run: cross build --release --target ${{ matrix.target }}
105105

106106
- name: Run tests (native only)
107-
if: matrix.cross == false && (github.event.inputs.test_type || 'full') != 'targets-only'
107+
if: ${{ matrix.cross == false && (github.event.inputs.test_type || 'full') != 'targets-only' }}
108108
run: cargo test --target ${{ matrix.target }} --verbose
109109

110110
- name: Verify binary functionality (native)
@@ -131,7 +131,7 @@ jobs:
131131
platform-specific-tests:
132132
name: Platform-Specific Tests
133133
runs-on: ${{ matrix.os }}
134-
if: (github.event.inputs.test_type || 'full') != 'targets-only'
134+
if: ${{ (github.event.inputs.test_type || 'full') != 'targets-only' }}
135135
strategy:
136136
fail-fast: false
137137
matrix:
@@ -237,7 +237,7 @@ jobs:
237237
integration-validation:
238238
name: Integration Validation
239239
runs-on: ubuntu-latest
240-
if: (github.event.inputs.test_type || 'full') == 'full'
240+
if: ${{ (github.event.inputs.test_type || 'full') == 'full' }}
241241
needs: [cross-compile-targets]
242242

243243
steps:

0 commit comments

Comments
 (0)