Skip to content

Commit

Permalink
Wrap condition in ${{ }}
Browse files Browse the repository at this point in the history
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.
  • Loading branch information
MiKom committed Jul 12, 2024
1 parent 73d93c6 commit b2fea82
Showing 1 changed file with 1 addition and 1 deletion.
2 changes: 1 addition & 1 deletion .github/workflows/build.yml
Original file line number Diff line number Diff line change
Expand Up @@ -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
Expand Down

0 comments on commit b2fea82

Please sign in to comment.