Skip to content

Commit 6a26f52

Browse files
committed
Fix workflow: use explicit != true for boolean inputs (unset != false)
1 parent 47592c6 commit 6a26f52

File tree

1 file changed

+13
-13
lines changed

1 file changed

+13
-13
lines changed

.github/workflows/dry-run.yml

Lines changed: 13 additions & 13 deletions
Original file line numberDiff line numberDiff line change
@@ -27,7 +27,7 @@ permissions:
2727
jobs:
2828
# ── Step 1: Lint (clang-format + cppcheck) ───────────────────
2929
lint:
30-
if: ${{ !inputs.skip_lint }}
30+
if: ${{ inputs.skip_lint != true }}
3131
runs-on: ubuntu-latest
3232
steps:
3333
- uses: actions/checkout@34e114876b0b11c390a56381ad16ebd13914f8d5 # v4
@@ -66,7 +66,7 @@ jobs:
6666
# No build needed — scans source files and vendored deps only.
6767
# Binary-level security (L2/L3/L4/L7) runs in smoke jobs per-platform.
6868
security-static:
69-
if: ${{ !inputs.skip_lint }}
69+
if: ${{ inputs.skip_lint != true }}
7070
runs-on: ubuntu-latest
7171
steps:
7272
- uses: actions/checkout@34e114876b0b11c390a56381ad16ebd13914f8d5 # v4
@@ -82,7 +82,7 @@ jobs:
8282

8383
# ── Step 1c: CodeQL SAST gate ────────────────────────────────
8484
codeql-gate:
85-
if: ${{ !inputs.skip_lint }}
85+
if: ${{ inputs.skip_lint != true }}
8686
runs-on: ubuntu-latest
8787
steps:
8888
- name: Wait for CodeQL on current commit (max 45 min)
@@ -158,7 +158,7 @@ jobs:
158158
# Linux: use system gcc — full ASan/UBSan support
159159
# Windows: MSYS2 MinGW GCC
160160
test-unix:
161-
if: ${{ !inputs.skip_tests && always() && (needs.lint.result == 'success' || needs.lint.result == 'skipped') }}
161+
if: ${{ inputs.skip_tests != true && always() && (needs.lint.result == 'success' || needs.lint.result == 'skipped') }}
162162
needs: [lint]
163163
strategy:
164164
fail-fast: false
@@ -192,7 +192,7 @@ jobs:
192192
run: scripts/test.sh CC=${{ matrix.cc }} CXX=${{ matrix.cxx }}
193193

194194
test-windows:
195-
if: ${{ !inputs.skip_tests && always() && (needs.lint.result == 'success' || needs.lint.result == 'skipped') }}
195+
if: ${{ inputs.skip_tests != true && always() && (needs.lint.result == 'success' || needs.lint.result == 'skipped') }}
196196
needs: [lint]
197197
runs-on: windows-latest
198198
steps:
@@ -214,7 +214,7 @@ jobs:
214214

215215
# ── Step 3: Build binaries (standard + UI, all OS) ──────────
216216
build-unix:
217-
if: ${{ !inputs.skip_builds && always() && (needs.test-unix.result == 'success' || needs.test-unix.result == 'skipped') && (needs.test-windows.result == 'success' || needs.test-windows.result == 'skipped') }}
217+
if: ${{ inputs.skip_builds != true && always() && (needs.test-unix.result == 'success' || needs.test-unix.result == 'skipped') && (needs.test-windows.result == 'success' || needs.test-windows.result == 'skipped') }}
218218
needs: [test-unix, test-windows]
219219
strategy:
220220
matrix:
@@ -287,7 +287,7 @@ jobs:
287287
path: "*.tar.gz"
288288

289289
build-windows:
290-
if: ${{ !inputs.skip_builds && always() && (needs.test-unix.result == 'success' || needs.test-unix.result == 'skipped') && (needs.test-windows.result == 'success' || needs.test-windows.result == 'skipped') }}
290+
if: ${{ inputs.skip_builds != true && always() && (needs.test-unix.result == 'success' || needs.test-unix.result == 'skipped') && (needs.test-windows.result == 'success' || needs.test-windows.result == 'skipped') }}
291291
needs: [test-unix, test-windows]
292292
runs-on: windows-latest
293293
steps:
@@ -338,7 +338,7 @@ jobs:
338338

339339
# ── Step 4: Smoke test every binary ─────────────────────────
340340
smoke-unix:
341-
if: ${{ !inputs.skip_builds }}
341+
if: ${{ inputs.skip_builds != true }}
342342
needs: [build-unix]
343343
strategy:
344344
fail-fast: false
@@ -437,7 +437,7 @@ jobs:
437437
echo "=== ClamAV: clean ==="
438438
439439
smoke-windows:
440-
if: ${{ !inputs.skip_builds }}
440+
if: ${{ inputs.skip_builds != true }}
441441
needs: [build-windows]
442442
strategy:
443443
fail-fast: false
@@ -511,7 +511,7 @@ jobs:
511511
512512
# ── Step 6: Soak tests (after smoke, per-platform) ──────────
513513
soak-quick:
514-
if: ${{ inputs.soak_level != 'none' && !inputs.skip_builds }}
514+
if: ${{ inputs.soak_level != 'none' && inputs.skip_builds != true }}
515515
needs: [build-unix]
516516
runs-on: ${{ matrix.os }}
517517
timeout-minutes: 30
@@ -561,7 +561,7 @@ jobs:
561561
retention-days: 14
562562

563563
soak-quick-windows:
564-
if: ${{ inputs.soak_level != 'none' && !inputs.skip_builds }}
564+
if: ${{ inputs.soak_level != 'none' && inputs.skip_builds != true }}
565565
needs: [build-windows]
566566
runs-on: windows-latest
567567
timeout-minutes: 30
@@ -597,7 +597,7 @@ jobs:
597597
retention-days: 14
598598

599599
soak-asan:
600-
if: ${{ inputs.soak_level == 'full' && !inputs.skip_builds }}
600+
if: ${{ inputs.soak_level == 'full' && inputs.skip_builds != true }}
601601
needs: [build-unix]
602602
runs-on: ${{ matrix.os }}
603603
timeout-minutes: 45
@@ -651,7 +651,7 @@ jobs:
651651
retention-days: 14
652652

653653
soak-asan-windows:
654-
if: ${{ inputs.soak_level == 'full' && !inputs.skip_builds }}
654+
if: ${{ inputs.soak_level == 'full' && inputs.skip_builds != true }}
655655
needs: [build-windows]
656656
runs-on: windows-latest
657657
timeout-minutes: 45

0 commit comments

Comments
 (0)