Skip to content

Commit 5ddec2a

Browse files
committed
Check OS using runner.os instead of matrix.os
This makes it easier to later update the OS verions Signed-off-by: Joe Richey <joerichey@google.com>
1 parent 8e4ecdb commit 5ddec2a

File tree

1 file changed

+3
-3
lines changed

1 file changed

+3
-3
lines changed

.github/workflows/tests.yml

Lines changed: 3 additions & 3 deletions
Original file line numberDiff line numberDiff line change
@@ -203,7 +203,7 @@ jobs:
203203
- uses: actions/checkout@v3
204204
- uses: dtolnay/rust-toolchain@stable
205205
- run: choco install wget
206-
if: matrix.os == 'windows-latest'
206+
if: runner.os == 'Windows'
207207
- name: Install precompiled wasm-pack
208208
shell: bash
209209
run: |
@@ -219,11 +219,11 @@ jobs:
219219
- name: Test (Chrome)
220220
run: wasm-pack test --headless --chrome --features=js,test-in-browser
221221
- name: Test (Edge)
222-
if: matrix.os == 'windows-latest'
222+
if: runner.os == 'Windows'
223223
run: wasm-pack test --headless --chrome --chromedriver $Env:EDGEWEBDRIVER\msedgedriver.exe --features=js,test-in-browser
224224
# Safari tests are broken: https://github.com/rustwasm/wasm-bindgen/issues/3004
225225
# - name: Test (Safari)
226-
# if: matrix.os == 'macos-12'
226+
# if: runner.os == 'macOS'
227227
# run: wasm-pack test --headless --safari --features=js,test-in-browser
228228
- name: Test (custom getrandom)
229229
run: wasm-pack test --node --features=custom

0 commit comments

Comments
 (0)