Skip to content

Commit deb7bb9

Browse files
justin808claude
andcommitted
Fix workflow syntax: remove matrix context from job-level if conditions
The matrix context is only available at step level, not job level. Removed the matrix-based filtering from job-level if conditions. Both matrix items will run on PRs, keeping the workflow simpler and fixing actionlint errors. 🤖 Generated with [Claude Code](https://claude.com/claude-code) Co-Authored-By: Claude <noreply@anthropic.com>
1 parent b0b49cd commit deb7bb9

File tree

4 files changed

+1
-8
lines changed

4 files changed

+1
-8
lines changed

.github/workflows/examples.yml

Lines changed: 1 addition & 4 deletions
Original file line numberDiff line numberDiff line change
@@ -37,10 +37,7 @@ jobs:
3737

3838
examples:
3939
needs: detect-changes
40-
# Run on master OR when generators changed on PR (but skip minimum deps on PR)
41-
if: |
42-
(github.ref == 'refs/heads/master' || needs.detect-changes.outputs.run_generators == 'true')
43-
&& (github.ref == 'refs/heads/master' || matrix.dependency-level == 'latest')
40+
if: github.ref == 'refs/heads/master' || needs.detect-changes.outputs.run_generators == 'true'
4441
strategy:
4542
fail-fast: false
4643
matrix:

.github/workflows/main.yml

Lines changed: 0 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -40,7 +40,6 @@ jobs:
4040
# Run on master OR when tests needed on PR (but skip minimum deps on PR)
4141
if: |
4242
(github.ref == 'refs/heads/master' || needs.detect-changes.outputs.run_dummy_tests == 'true')
43-
&& (github.ref == 'refs/heads/master' || matrix.dependency-level == 'latest')
4443
strategy:
4544
matrix:
4645
include:
@@ -123,7 +122,6 @@ jobs:
123122
# Run on master OR when tests needed on PR (but skip minimum deps on PR)
124123
if: |
125124
(github.ref == 'refs/heads/master' || needs.detect-changes.outputs.run_dummy_tests == 'true')
126-
&& (github.ref == 'refs/heads/master' || matrix.dependency-level == 'latest')
127125
strategy:
128126
fail-fast: false
129127
matrix:

.github/workflows/package-js-tests.yml

Lines changed: 0 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -44,7 +44,6 @@ jobs:
4444
# Run on master OR when JS tests needed on PR (but skip Node 20 on PR)
4545
if: |
4646
(github.ref == 'refs/heads/master' || needs.detect-changes.outputs.run_js_tests == 'true')
47-
&& (github.ref == 'refs/heads/master' || matrix.node-version == '22')
4847
strategy:
4948
matrix:
5049
include:

.github/workflows/rspec-package-specs.yml

Lines changed: 0 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -44,7 +44,6 @@ jobs:
4444
# Run on master OR when Ruby tests needed on PR (but skip minimum deps on PR)
4545
if: |
4646
(github.ref == 'refs/heads/master' || needs.detect-changes.outputs.run_ruby_tests == 'true')
47-
&& (github.ref == 'refs/heads/master' || matrix.dependency-level == 'latest')
4847
strategy:
4948
fail-fast: false
5049
matrix:

0 commit comments

Comments
 (0)