Skip to content

Commit 8a15f80

Browse files
committed
Fixed wrong usage of matrix.vs. That label is os version, not vs version
1 parent 0e79279 commit 8a15f80

File tree

1 file changed

+14
-11
lines changed

1 file changed

+14
-11
lines changed

.github/workflows/windows.yml

Lines changed: 14 additions & 11 deletions
Original file line numberDiff line numberDiff line change
@@ -25,19 +25,22 @@ jobs:
2525
strategy:
2626
matrix:
2727
include:
28-
- vc: 2015
29-
vs: 2019
28+
- os: 2019
29+
vc: 2015
3030
vcvars: '10.0.14393.0 -vcvars_ver=14.0' # The oldest Windows 10 SDK w/ VC++ 2015 toolset (v140)
3131
test_task: check
32-
- vs: 2019
32+
- os: 2019
33+
vc: 2019
3334
test_task: check
34-
- vs: 2019
35+
- os: 2019
36+
vc: 2019
3537
test_task: test-bundled-gems
36-
# - vs: 2022
38+
# - os: 2022
39+
# vc: 2022
3740
# test_task: check
3841
fail-fast: false
3942

40-
runs-on: windows-${{ matrix.vs < 2022 && '2019' || matrix.vs }}
43+
runs-on: windows-${{ matrix.os < 2022 && '2019' || matrix.os }}
4144

4245
if: >-
4346
${{!(false
@@ -49,11 +52,11 @@ jobs:
4952
|| (github.event_name == 'push' && github.event.pull_request.user.login == 'dependabot[bot]')
5053
)}}
5154
52-
name: VisualStudio ${{ matrix.vc || matrix.vs }} (${{ matrix.test_task }})
55+
name: Windows ${{ matrix.os }}/Visual C++ ${{ matrix.vc }} (${{ matrix.test_task }})
5356

5457
env:
5558
GITPULLOPTIONS: --no-tags origin ${{ github.ref }}
56-
OS_VER: windows-${{ matrix.vs < 2022 && '2019' || matrix.vs }}
59+
OS_VER: windows-${{ matrix.os < 2022 && '2019' || matrix.os }}
5760
VCPKG_DEFAULT_TRIPLET: ${{ matrix.target || 'x64' }}-windows
5861

5962
steps:
@@ -123,7 +126,7 @@ jobs:
123126
# %TEMP% is inconsistent with %TMP% and test-all expects they are consistent.
124127
# https://github.com/actions/virtual-environments/issues/712#issuecomment-613004302
125128
run: |
126-
::- Set up VC ${{ matrix.vc || matrix.vs }}
129+
::- Set up VC ${{ matrix.vc }}
127130
set vswhere="%ProgramFiles(x86)%\Microsoft Visual Studio\Installer\vswhere.exe"
128131
for /f "delims=;" %%I in ('%vswhere% -latest -property installationPath') do (
129132
set VCVARS="%%I\VC\Auxiliary\Build\vcvars64.bat"
@@ -179,7 +182,7 @@ jobs:
179182
- name: Set up Launchable
180183
uses: ./.github/actions/launchable/setup
181184
with:
182-
os: windows-${{ matrix.vs < 2022 && '2019' || matrix.vs }}
185+
os: windows-${{ matrix.os < 2022 && '2019' || matrix.os }}
183186
launchable-token: ${{ secrets.LAUNCHABLE_TOKEN }}
184187
builddir: build
185188
srcdir: src
@@ -194,7 +197,7 @@ jobs:
194197

195198
- uses: ./.github/actions/slack
196199
with:
197-
label: VS${{ matrix.vc || matrix.vs }} / ${{ matrix.test_task || 'check' }}
200+
label: Windows ${{ matrix.os }} / VC ${{ matrix.vc }} / ${{ matrix.test_task || 'check' }}
198201
SLACK_WEBHOOK_URL: ${{ secrets.SIMPLER_ALERTS_URL }} # ruby-lang slack: ruby/simpler-alerts-bot
199202
if: ${{ failure() }}
200203

0 commit comments

Comments
 (0)