Skip to content

Commit 84b6708

Browse files
authored
Merge pull request #6 from goddessfreya/goddessfreya-patch-1
Add clang testing
2 parents fef5593 + 1a0243e commit 84b6708

File tree

2 files changed

+13
-371
lines changed

2 files changed

+13
-371
lines changed

.github/workflows/nodejs.yml

+13-6
Original file line numberDiff line numberDiff line change
@@ -9,23 +9,30 @@ jobs:
99
matrix:
1010
node-version: [8.x, 10.x, 12.x]
1111
platform:
12-
- { os: windows-latest, hunspell: true }
12+
- { os: windows-latest, force-hunspell: true }
1313
- { os: windows-latest }
14-
- { os: macos-latest, hunspell: true }
14+
- { os: macos-latest, force-hunspell: true }
1515
- { os: macos-latest }
16-
- { os: ubuntu-latest }
16+
- { os: ubuntu-latest, use-clang: true }
1717

1818
runs-on: ${{ matrix.platform.os }}
19-
19+
env:
20+
SPELLCHECKER_PREFER_HUNSPELL: ${{ matrix.platform.hunspell }}
21+
2022
steps:
2123
- uses: actions/checkout@v1
2224
- name: Use Node.js ${{ matrix.node-version }}
2325
uses: actions/setup-node@v1
2426
with:
2527
node-version: ${{ matrix.node-version }}
28+
- name: npm install, build, and test (clang)
29+
if: matrix.platform.use-clang == true
30+
run: |
31+
export CC="$(which clang)"
32+
export CPP="$(which clang++)"
33+
npm install
34+
npm test
2635
- name: npm install, build, and test
2736
run: |
2837
npm install
2938
npm test
30-
env:
31-
SPELLCHECKER_PREFER_HUNSPELL: ${{ matrix.platform.hunspell }}

0 commit comments

Comments
 (0)