From b2fea82e31516a3971a10c910ba64c733d75875e Mon Sep 17 00:00:00 2001 From: =?UTF-8?q?Mi=C5=82osz=20Kosobucki?= Date: Thu, 11 Jul 2024 17:55:11 +0200 Subject: [PATCH] Wrap condition in ${{ }} As @dantti suggested, it's a good habit to wrap the condition in ${{}} as e.g. the negation mark (!) is a special YAML character and if there's a problem with the tested variable it may evaluate to weird stuff and be hard to debug. Let's be defensive. --- .github/workflows/build.yml | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/.github/workflows/build.yml b/.github/workflows/build.yml index 306655d..51adc88 100644 --- a/.github/workflows/build.yml +++ b/.github/workflows/build.yml @@ -32,7 +32,7 @@ jobs: uses: actions/checkout@v4 - name: Install ninja-build tool - if: runner.os != 'Linux' + if: ${{ runner.os != 'Linux' }} uses: turtlesec-no/get-ninja@main - name: Make sure MSVC is found when Ninja generator is in use