Skip to content
This repository was archived by the owner on Dec 15, 2022. It is now read-only.

Commit d17cf27

Browse files
authored
Update nodejs.yml
1 parent 81c5ae6 commit d17cf27

File tree

1 file changed

+9
-9
lines changed

1 file changed

+9
-9
lines changed

.github/workflows/nodejs.yml

+9-9
Original file line numberDiff line numberDiff line change
@@ -13,7 +13,6 @@ jobs:
1313
- { os: windows-latest }
1414
- { os: macos-latest, force-hunspell: true }
1515
- { os: macos-latest }
16-
- { os: ubuntu-latest }
1716
- { os: ubuntu-latest, use-clang: true }
1817

1918
runs-on: ${{ matrix.platform.os }}
@@ -26,14 +25,15 @@ jobs:
2625
uses: actions/setup-node@v1
2726
with:
2827
node-version: ${{ matrix.node-version }}
29-
- name: Clang env
28+
- name: npm install, build, and test (clang)
3029
if: matrix.platform.use-clang == true
3130
run: |
3231
export CC="$(which clang)"
3332
export CPP="$(which clang++)"
33+
npm install
34+
npm test
3435
- name: npm install, build, and test
3536
run: |
36-
echo "Using $CC"
3737
npm install
3838
npm test
3939
@@ -46,7 +46,6 @@ jobs:
4646
platform:
4747
- { os: windows-latest }
4848
- { os: macos-latest }
49-
- { os: ubuntu-latest }
5049
- { os: ubuntu-latest, use-clang: true }
5150

5251
runs-on: ${{ matrix.platform.os }}
@@ -61,15 +60,16 @@ jobs:
6160
uses: actions/setup-python@v1
6261
with:
6362
python-version: ${{ matrix.python-version }}
64-
- name: Clang env
63+
- name: node-gyp install
64+
run: |
65+
npm install -g node-gyp
66+
- name: node-gyp configure & build (clang)
6567
if: matrix.platform.use-clang == true
6668
run: |
6769
export CC="$(which clang)"
6870
export CPP="$(which clang++)"
69-
- name: node-gyp install
70-
run: |
71-
echo "Using $CC"
72-
npm install -g node-gyp
71+
node-gyp configure
72+
node-gyp build
7373
- name: node-gyp configure & build
7474
run: |
7575
node-gyp configure

0 commit comments

Comments
 (0)