From b546bf6dc9038c1f83af3d982dbd598d6a50d16c Mon Sep 17 00:00:00 2001
From: Sammy Sidhu <samster25@users.noreply.github.com>
Date: Sun, 24 Sep 2023 16:16:47 -0700
Subject: [PATCH] enable all

---
 .github/workflows/python-package.yml | 9 ++++++---
 .github/workflows/python-publish.yml | 4 ++--
 2 files changed, 8 insertions(+), 5 deletions(-)

diff --git a/.github/workflows/python-package.yml b/.github/workflows/python-package.yml
index 8269be72cf..84537dada4 100644
--- 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
index f9f87e9de7..10044cf067 100644
--- 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: