Skip to content

Commit 35ccbec

Browse files
authored
fix(ci): gate node suite jobs by changed paths
1 parent b623fcf commit 35ccbec

1 file changed

Lines changed: 8 additions & 9 deletions

File tree

.github/workflows/test.yml

Lines changed: 8 additions & 9 deletions
Original file line numberDiff line numberDiff line change
@@ -84,34 +84,33 @@ jobs:
8484
env:
8585
node-version: lts/*
8686
redis-version: 7-alpine
87-
run_smoke: ${{ needs.changes.outputs.node == 'true' || needs.changes.outputs.lua == 'true' }}
8887

8988
steps:
9089
- name: Skip smoke run when node/lua inputs are unchanged
91-
if: ${{ env.run_smoke != 'true' }}
90+
if: ${{ needs.changes.outputs.node != 'true' && needs.changes.outputs.lua != 'true' }}
9291
run: echo "No node/lua changes detected; skipping smoke steps."
9392
- name: Checkout repository
94-
if: ${{ env.run_smoke == 'true' }}
93+
if: ${{ needs.changes.outputs.node == 'true' || needs.changes.outputs.lua == 'true' }}
9594
uses: actions/checkout@de0fac2e4500dabe0009e67214ff5f5447ce83dd # v6
9695
- name: Use Node.js
97-
if: ${{ env.run_smoke == 'true' }}
96+
if: ${{ needs.changes.outputs.node == 'true' || needs.changes.outputs.lua == 'true' }}
9897
uses: actions/setup-node@48b55a011bda9f5d6aeb4c2d9c7362e8dae4041e # v6
9998
with:
10099
node-version: ${{ env.node-version }}
101100
cache: 'yarn'
102101
- name: Start Redis
103-
if: ${{ env.run_smoke == 'true' }}
102+
if: ${{ needs.changes.outputs.node == 'true' || needs.changes.outputs.lua == 'true' }}
104103
uses: supercharge/redis-github-action@bc274cb7238cd63a45029db04ee48c07a72609fd # 1.8.1
105104
with:
106105
redis-version: ${{ env.redis-version }}
107-
- if: ${{ env.run_smoke == 'true' }}
106+
- if: ${{ needs.changes.outputs.node == 'true' || needs.changes.outputs.lua == 'true' }}
108107
run: yarn install --ignore-engines --frozen-lockfile --non-interactive
109108
- name: Run ESLint
110-
if: ${{ env.run_smoke == 'true' }}
109+
if: ${{ needs.changes.outputs.node == 'true' || needs.changes.outputs.lua == 'true' }}
111110
run: yarn lint
112-
- if: ${{ env.run_smoke == 'true' }}
111+
- if: ${{ needs.changes.outputs.node == 'true' || needs.changes.outputs.lua == 'true' }}
113112
run: yarn build
114-
- if: ${{ env.run_smoke == 'true' }}
113+
- if: ${{ needs.changes.outputs.node == 'true' || needs.changes.outputs.lua == 'true' }}
115114
run: yarn test:smoke
116115

117116
# Primary ioredis test run on a single (current LTS) Node version.

0 commit comments

Comments
 (0)