Skip to content

Commit 74701dc

Browse files
committed
Auto merge of rust-lang#3547 - RalfJung:ci, r=RalfJung
CI: no need to surround if: condition in expansion braces That seems to be implicit for `if:` (but interestingly, redundant braces are tolerated).
2 parents 692b769 + aa71f9b commit 74701dc

File tree

1 file changed

+3
-3
lines changed
  • src/tools/miri/.github/workflows

1 file changed

+3
-3
lines changed

src/tools/miri/.github/workflows/ci.yml

Lines changed: 3 additions & 3 deletions
Original file line numberDiff line numberDiff line change
@@ -61,7 +61,7 @@ jobs:
6161
restore-keys: cargo-${{ runner.os }}-reset20240425
6262

6363
- name: Install tools
64-
if: ${{ steps.cache.outputs.cache-hit != 'true' }}
64+
if: steps.cache.outputs.cache-hit != 'true'
6565
run: cargo install -f rustup-toolchain-install-master hyperfine
6666

6767
- name: Install miri toolchain
@@ -81,7 +81,7 @@ jobs:
8181
# The `style` job only runs on Linux; this makes sure the Windows-host-specific
8282
# code is also covered by clippy.
8383
- name: Check clippy
84-
if: ${{ matrix.os == 'windows-latest' }}
84+
if: matrix.os == 'windows-latest'
8585
run: ./miri clippy -- -D warnings
8686

8787
- name: Test Miri
@@ -117,7 +117,7 @@ jobs:
117117
restore-keys: cargo-${{ runner.os }}-reset20240331
118118

119119
- name: Install rustup-toolchain-install-master
120-
if: ${{ steps.cache.outputs.cache-hit != 'true' }}
120+
if: steps.cache.outputs.cache-hit != 'true'
121121
run: cargo install -f rustup-toolchain-install-master
122122

123123
- name: Install "master" toolchain

0 commit comments

Comments
 (0)