Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Update for node Node 22, update AppVeyor and tests #712

Merged
merged 26 commits into from
Jun 29, 2024
Merged
Show file tree
Hide file tree
Changes from 20 commits
Commits
Show all changes
26 commits
Select commit Hold shift + click to select a range
bbfd7b2
Fixing issue 647
ronilan May 4, 2022
e26d438
run.util.js: Test building with msvs 2022
cclauss Apr 11, 2024
d175f48
Update appveyor.yml
cclauss Apr 11, 2024
cae1f89
Update appveyor.yml
cclauss Apr 11, 2024
73050e4
Update appveyor.yml
cclauss Apr 11, 2024
b33fee5
Update appveyor.yml
cclauss Apr 11, 2024
aad20cd
Update appveyor.yml
cclauss Apr 11, 2024
d866f51
Update appveyor.yml
cclauss Apr 11, 2024
97e4670
Update appveyor.yml
cclauss Apr 11, 2024
70c1dca
Update appveyor.yml
cclauss Apr 11, 2024
13a51d8
Update appveyor.yml
cclauss Apr 11, 2024
fbd3cce
mapbox/node-pre-gyp#710
cclauss Apr 11, 2024
2ef0096
update abi_crosswalk.json
acalcutt Apr 11, 2024
77ee55c
Merge branch 'mapbox:master' into Fix-Support-Node-18-Target-647
ronilan Apr 11, 2024
95264e7
Merge remote-tracking branch 'cclauss/patch-1' into node21
acalcutt Apr 11, 2024
56334ee
Update appveyor.yml for node 20 support
acalcutt Apr 11, 2024
88be0a2
Merge remote-tracking branch 'ronilan/Fix-Support-Node-18-Target-647'…
acalcutt Apr 11, 2024
7804f5c
test: show command
acalcutt Apr 11, 2024
17ee1cc
Revert "test: show command"
acalcutt Apr 11, 2024
e5a9524
add line ending to abi_crosswalk.json
acalcutt Apr 11, 2024
9dec1f1
add node 21 to appveyor.yml
acalcutt Apr 11, 2024
e33ff0c
Update appveyor.yml
acalcutt Apr 12, 2024
aaabb80
npm run update-crosswalk
acalcutt Apr 19, 2024
a8aa868
npm run update-crosswalk for node 22
acalcutt Jun 27, 2024
9acee69
change node 21 to node 22 in appveyor.yml
acalcutt Jun 27, 2024
5396d8a
Merge remote-tracking branch 'upstream/master' into node21
acalcutt Jun 29, 2024
File filter

Filter by extension

Filter by extension

Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
24 changes: 13 additions & 11 deletions appveyor.yml
Original file line number Diff line number Diff line change
@@ -1,10 +1,11 @@
os: Visual Studio 2015
image: # http://www.appveyor.com/docs/appveyor-yml
- Visual Studio 2022

environment:
matrix:
- nodejs_version: 10
- nodejs_version: 12
- nodejs_version: 14
matrix: # https://github.com/nodejs/release#release-schedule
- nodejs_version: 16
- nodejs_version: 18
- nodejs_version: 20
Copy link
Contributor Author

@acalcutt acalcutt Apr 12, 2024

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

I made this change, but i'm not sure it is corrent since it seemed only lts releases were listed here before


platform:
acalcutt marked this conversation as resolved.
Show resolved Hide resolved
- x64
Expand All @@ -13,17 +14,18 @@ platform:
shallow_clone: true

install:
- ps: Install-Product node $env:nodejs_version $env:Platform
- ps: Install-Product node $env:nodejs_version $env:platform
- ps: Set-ExecutionPolicy Unrestricted -Scope CurrentUser -Force
- npm config get
- node --version
- npm --version
- node -e "console.log(process.arch);"
- IF /I "%PLATFORM%" == "x64" set PATH=C:\Python27-x64;%PATH%
- IF /I "%PLATFORM%" == "x86" SET PATH=C:\python27;%PATH%
- IF /I "%PLATFORM%" == "x64" CALL "C:\Program Files (x86)\Microsoft Visual Studio 14.0\VC\vcvarsall.bat" amd64
- IF /I "%PLATFORM%" == "x86" CALL "C:\Program Files (x86)\Microsoft Visual Studio 14.0\VC\vcvarsall.bat" x86
- node -e "console.log(process.arch);" # 'x64' or 'ia32'
# Python >= v3.12 requires node-gyp >= v10
- IF /I "%PLATFORM%" == "x64" set PATH=C:\Python311-x64;%PATH%
- IF /I "%PLATFORM%" == "x86" SET PATH=C:\python311;%PATH%
- npm config list
- npm ci
acalcutt marked this conversation as resolved.
Show resolved Hide resolved
# - npm audit # TODO: Fix mapbox/node-pre-gyp#705 and enable this line
- npm test

build: off
Expand Down
Loading