Skip to content

Commit 9dfbf10

Browse files
ci: specify target platform in build commands\n\n- Replace npm run build:native with explicit target platform builds\n- Each CI job now only builds for its specific target\n- Avoid attempting to build for all platforms at once
1 parent 561c0e6 commit 9dfbf10

File tree

1 file changed

+20
-9
lines changed

1 file changed

+20
-9
lines changed

.github/workflows/CI.yml

Lines changed: 20 additions & 9 deletions
Original file line numberDiff line numberDiff line change
@@ -22,49 +22,60 @@ jobs:
2222
- host: macos-latest
2323
target: x86_64-apple-darwin
2424
build: |
25-
npm run build
25+
npm install
26+
npx @napi-rs/cli build --target x86_64-apple-darwin
2627
strip -x *.node
2728
- host: macos-latest
2829
target: aarch64-apple-darwin
2930
build: |
30-
npm run build --target aarch64-apple-darwin
31+
npm install
32+
npx @napi-rs/cli build --target aarch64-apple-darwin
3133
strip -x *.node
3234
- host: windows-latest
3335
target: x86_64-pc-windows-msvc
34-
build: yarn build
36+
build: |
37+
npm install
38+
npx @napi-rs/cli build --target x86_64-pc-windows-msvc
3539
- host: windows-latest
3640
target: aarch64-pc-windows-msvc
37-
build: npm run build --target aarch64-pc-windows-msvc
41+
build: |
42+
npm install
43+
npx @napi-rs/cli build --target aarch64-pc-windows-msvc
3844
- host: ubuntu-latest
3945
target: x86_64-unknown-linux-gnu
4046
docker: ghcr.io/napi-rs/napi-rs/nodejs-rust:lts-debian
4147
build: |
4248
set -e &&
43-
npm run build --target x86_64-unknown-linux-gnu &&
49+
npm install &&
50+
npx @napi-rs/cli build --target x86_64-unknown-linux-gnu &&
4451
strip *.node
4552
- host: ubuntu-latest
4653
target: aarch64-unknown-linux-gnu
4754
docker: ghcr.io/napi-rs/napi-rs/nodejs-rust:lts-debian-aarch64
4855
build: |
4956
set -e &&
50-
npm run build --target aarch64-unknown-linux-gnu &&
57+
npm install &&
58+
npx @napi-rs/cli build --target aarch64-unknown-linux-gnu &&
5159
aarch64-unknown-linux-gnu-strip *.node
5260
- host: ubuntu-latest
5361
target: armv7-unknown-linux-gnueabihf
5462
docker: ghcr.io/napi-rs/napi-rs/nodejs-rust:lts-debian-arm
5563
build: |
5664
set -e &&
57-
npm run build --target armv7-unknown-linux-gnueabihf &&
65+
npm install &&
66+
npx @napi-rs/cli build --target armv7-unknown-linux-gnueabihf &&
5867
arm-linux-gnueabihf-strip *.node
5968
- host: ubuntu-latest
6069
target: aarch64-linux-android
6170
build: |
62-
npm run build --target aarch64-linux-android
71+
npm install
72+
npx @napi-rs/cli build --target aarch64-linux-android
6373
${ANDROID_NDK_LATEST_HOME}/toolchains/llvm/prebuilt/linux-x86_64/bin/llvm-strip *.node
6474
- host: ubuntu-latest
6575
target: armv7-linux-androideabi
6676
build: |
67-
npm run build --target armv7-linux-androideabi
77+
npm install
78+
npx @napi-rs/cli build --target armv7-linux-androideabi
6879
${ANDROID_NDK_LATEST_HOME}/toolchains/llvm/prebuilt/linux-x86_64/bin/llvm-strip *.node
6980
7081
name: stable - ${{ matrix.settings.target }} - node@18

0 commit comments

Comments
 (0)