enable all

Eventual-Inc · Sep 24, 2023 · b546bf6 · b546bf6
1 parent 5070e68
commit b546bf6
Show file tree

Hide file tree

Showing 2 changed files with 8 additions and 5 deletions.
diff --git a/.github/workflows/python-package.yml b/.github/workflows/python-package.yml
@@ -6,15 +6,15 @@ name: daft
 on:
   push:
     branches: [main]
-  # pull_request:
-  #   branches: [main]
+  pull_request:
+    branches: [main]
 
 env:
   DAFT_ANALYTICS_ENABLED: '0'
 
 jobs:
   unit-tests-with-coverage:
-    runs-on: ubuntu-latest
+    runs-on: ${{ matrix.os }}-latest
     timeout-minutes: 15
     strategy:
       fail-fast: false
@@ -23,12 +23,15 @@ jobs:
         daft-runner: [py, ray]
         new-query-planner: [1, 0]
         pyarrow-version: [6.0.1, 12.0]
+        os: [ubuntu, windows]
         exclude:
         - daft-runner: ray
           pyarrow-version: 6.0.1
         - daft-runner: py
           python-version: '3.10'
           pyarrow-version: 6.0.1
+        - os: windows
+          pyarrow-version: 12.0
 
     steps:
     - uses: actions/checkout@v4

diff --git a/.github/workflows/python-publish.yml b/.github/workflows/python-publish.yml
@@ -37,7 +37,7 @@ jobs:
         runs: [windows-latest]
         compile_arch: [x86_64, aarch64]
         exclude:
-        - os: windows-latest
+        - runs: windows-latest
           compile_arch: aarch64
     steps:
     - uses: actions/checkout@v4
@@ -102,7 +102,7 @@ jobs:
         pip install -r requirements-dev.txt dist/${{ env.PACKAGE_NAME }}-*x86_64*.whl --force-reinstall
         rd -r daft
         pytest -v
-  
+
     - name: Upload wheels
       uses: actions/upload-artifact@v3
       with: