Skip to content

Commit 5ba04ca

Browse files
committed
Don't prebuild on Node 10-14 in CI
1 parent ef43960 commit 5ba04ca

File tree

2 files changed

+13
-27
lines changed

2 files changed

+13
-27
lines changed

.github/workflows/build.yml

Lines changed: 13 additions & 20 deletions
Original file line numberDiff line numberDiff line change
@@ -51,34 +51,27 @@ jobs:
5151
- macos-latest
5252
- ubuntu-latest
5353
node:
54-
- 10
55-
- 12
56-
- 14
57-
- 16
58-
- 18
59-
- 20
60-
exclude:
54+
# Node 10-14 fails to prebuild for Node 18+
55+
#
6156
# macos-latest's Python version is too new to work with the node-gyp
6257
# bundled by prebuild and the "overrides" in package.json for node-gyp
6358
# doesn't work before npm 8.3 (Node 16)
64-
- os: macos-latest
65-
node: 10
66-
- os: macos-latest
67-
node: 12
68-
- os: macos-latest
69-
node: 14
70-
# Similar issue but in this case the error is more direct:
7159
#
60+
# ubuntu-latest has a similar issue but in this case the error is more direct:
7261
# #error "It looks like you are building this native module without
7362
# using the right config.gypi. This normally means that you need to
7463
# update electron-rebuild (>=3.2.8) or node-gyp (>=8.4.0) if you're
7564
# building modules directly."
76-
- os: ubuntu-latest
77-
node: 10
78-
- os: ubuntu-latest
79-
node: 12
80-
- os: ubuntu-latest
81-
node: 14
65+
#
66+
# windows-2019 fails with
67+
# gyp: name 'llvm_version' is not defined while evaluating condition
68+
# 'llvm_version=="0.0"' in binding.gyp while trying to load binding.gyp
69+
# - 10
70+
# - 12
71+
# - 14
72+
- 16
73+
- 18
74+
- 20
8275
fail-fast: false
8376
name: Prebuild with Node ${{ matrix.node }} on ${{ matrix.os }}
8477
runs-on: ${{ matrix.os }}

binding.gyp

Lines changed: 0 additions & 7 deletions
Original file line numberDiff line numberDiff line change
@@ -43,13 +43,6 @@
4343
'RuntimeLibrary': 0,
4444
},
4545
},
46-
'variables': {
47-
# fix this error when prebuilding for Node 18 on Node 14 or older
48-
#
49-
# gyp: name 'llvm_version' is not defined while evaluating condition
50-
# 'llvm_version=="0.0"' in binding.gyp while trying to load binding.gyp
51-
'llvm_version': 0,
52-
}
5346
}]
5447
],
5548
},

0 commit comments

Comments
 (0)