test on windows and mac (in addition to existing linux tests)

Closes #192
express-rate-limit · Feb 28, 2021 · 7d3f979 · 7d3f979
1 parent 8c61e4e
commit 7d3f979
Show file tree

Hide file tree

Showing 2 changed files with 8 additions and 5 deletions.
diff --git a/.github/workflows/publish.yml b/.github/workflows/publish.yml
@@ -10,7 +10,7 @@ on:
 
 jobs:
   build:
-    runs-on: ubuntu-latest
+    runs-on: [ubuntu-latest, windows-latest, macos-latest]
     steps:
       - uses: actions/checkout@v2
       - uses: actions/setup-node@v1

diff --git a/.github/workflows/test.yml b/.github/workflows/test.yml
@@ -7,16 +7,19 @@ on: [push, pull_request]
 
 jobs:
   build:
-
-    runs-on: ubuntu-latest
-
     strategy:
       matrix:
         node-version: [10.x, 12.x, 14.x]
+        os: [ubuntu-latest, macos-latest, windows-latest]
+    runs-on: ${{ matrix.os }}
 
     steps:
+    # this is needed because otherwise it converts LF to CRLF on windows and then the linter fails the build
+    - name: Disable git autocrlf
+      run: |
+        git config --global core.autocrlf false
     - uses: actions/checkout@v2
-    - name: Use Node.js ${{ matrix.node-version }}
+    - name: Use Node.js ${{ matrix.node-version }} on ${{ matrix.os }}
       uses: actions/setup-node@v1
       with:
         node-version: ${{ matrix.node-version }}