Skip to content

Commit 7c27294

Browse files
Explicitly guard optional matrix steps in CI workflow (#89)
* Initial plan * Guard optional matrix steps with explicit non-empty conditions Co-authored-by: JohannesLorenz <1042576+JohannesLorenz@users.noreply.github.com> * Use truthy check for optional matrix fields instead of != '' comparison Co-authored-by: JohannesLorenz <1042576+JohannesLorenz@users.noreply.github.com> --------- Co-authored-by: copilot-swe-agent[bot] <198982749+Copilot@users.noreply.github.com> Co-authored-by: JohannesLorenz <1042576+JohannesLorenz@users.noreply.github.com>
1 parent fed1a43 commit 7c27294

1 file changed

Lines changed: 3 additions & 3 deletions

File tree

.github/workflows/ccpp.yml

Lines changed: 3 additions & 3 deletions
Original file line numberDiff line numberDiff line change
@@ -83,13 +83,13 @@ jobs:
8383
with:
8484
fetch-depth: 0
8585
- name: update system
86-
if: ${{ matrix.config.enabled == 1 }}
86+
if: ${{ matrix.config.enabled == 1 && matrix.config.update }}
8787
run: ${{ matrix.config.update }}
8888
- name: install deps (1)
89-
if: ${{ matrix.config.enabled == 1 }}
89+
if: ${{ matrix.config.enabled == 1 && matrix.config.deps1 }}
9090
run: ${{ matrix.config.deps1 }}
9191
- name: install deps (2)
92-
if: ${{ matrix.config.enabled == 1 }}
92+
if: ${{ matrix.config.enabled == 1 && matrix.config.deps2 }}
9393
run: ${{ matrix.config.deps2 }}
9494
- name: create build directory
9595
if: ${{ matrix.config.enabled == 1 }}

0 commit comments

Comments
 (0)