Skip to content

Commit

Permalink
Windows 2025 currently ships with Visual Studio 2022
Browse files Browse the repository at this point in the history
  • Loading branch information
cclauss committed Jan 7, 2025
1 parent 44db885 commit aa2aa3e
Show file tree
Hide file tree
Showing 3 changed files with 6 additions and 9 deletions.
4 changes: 2 additions & 2 deletions .github/workflows/tests.yml
Original file line number Diff line number Diff line change
Expand Up @@ -132,13 +132,13 @@ jobs:
- name: Run Python Tests
run: python -m pytest
- name: Run Tests (macOS or Linux)
if: "!startsWith(matrix.os, 'windows')"
if: runner.os != 'Windows'
shell: bash
run: npm test --python="${pythonLocation}/python"
env:
FULL_TEST: ${{ (matrix.node == '22.x' && matrix.python == '3.13') && '1' || '0' }}
- name: Run Tests (Windows)
if: startsWith(matrix.os, 'windows')
if: runner.os == 'Windows'
shell: bash # Building wasm on Windows requires using make generator, it only works in bash
run: npm run test --python="${pythonLocation}\\python.exe"
env:
Expand Down
2 changes: 1 addition & 1 deletion .github/workflows/visual-studio.yml
Original file line number Diff line number Diff line change
Expand Up @@ -22,7 +22,7 @@ jobs:
- os: windows-2022
msvs-version: 2022
- os: windows-2025
msvs-version: 2025
msvs-version: 2022 # Fix this when Visual Studio 2025 is released
runs-on: ${{ matrix.os }}
steps:
- name: Checkout Repository
Expand Down
9 changes: 3 additions & 6 deletions lib/find-visualstudio.js
Original file line number Diff line number Diff line change
Expand Up @@ -385,14 +385,11 @@ class VisualStudioFinder {
ret.versionYear = 2019
return ret
}
if (ret.versionMajor === 17) {
// Fix this when Visual Studio 2025 ships
if ([17, 18].includes(versionMajor)) {

Check failure on line 389 in lib/find-visualstudio.js

View workflow job for this annotation

GitHub Actions / Lint JS

'versionMajor' is not defined
ret.versionYear = 2022
return ret
}
if (ret.versionMajor === 18) {
ret.versionYear = 2025
return ret
}
this.log.silly('- unsupported version:', ret.versionMajor)
return {}
}
Expand Down Expand Up @@ -461,7 +458,7 @@ class VisualStudioFinder {
} else if (versionYear === 2022) {
return 'v143'
} else if (versionYear === 2025) {
return 'v144'
return 'v143' // Fix this when Visual Studio 2025 ships
}
this.log.silly('- invalid versionYear:', versionYear)
return null
Expand Down

0 comments on commit aa2aa3e

Please sign in to comment.