Merge pull request #6 from goddessfreya/goddessfreya-patch-1

Add clang testing
atom · Dec 20, 2019 · 84b6708 · 84b6708
2 parents fef5593 + 1a0243e
commit 84b6708
Show file tree

Hide file tree

Showing 2 changed files with 13 additions and 371 deletions.
diff --git a/.github/workflows/nodejs.yml b/.github/workflows/nodejs.yml
@@ -9,23 +9,30 @@ jobs:
       matrix:
         node-version: [8.x, 10.x, 12.x]
         platform: 
-          - { os: windows-latest, hunspell: true }
+          - { os: windows-latest, force-hunspell: true }
           - { os: windows-latest }
-          - { os: macos-latest, hunspell: true }  
+          - { os: macos-latest, force-hunspell: true }  
           - { os: macos-latest }  
-          - { os: ubuntu-latest }
+          - { os: ubuntu-latest, use-clang: true }
 
     runs-on: ${{ matrix.platform.os }}
-
+    env:
+      SPELLCHECKER_PREFER_HUNSPELL: ${{ matrix.platform.hunspell }}
+
     steps:
     - uses: actions/checkout@v1
     - name: Use Node.js ${{ matrix.node-version }}
       uses: actions/setup-node@v1
       with:
         node-version: ${{ matrix.node-version }}
+    - name: npm install, build, and test (clang)
+      if: matrix.platform.use-clang == true
+      run: |
+        export CC="$(which clang)"
+        export CPP="$(which clang++)"
+        npm install
+        npm test
     - name: npm install, build, and test
       run: |
         npm install
         npm test
-      env:
-        SPELLCHECKER_PREFER_HUNSPELL: ${{ matrix.platform.hunspell }}