CI ordering was wrong and maturin needed uv option

apache · Jan 14, 2025 · d299124 · d299124
1 parent e27831d
commit d299124
Show file tree

Hide file tree

Showing 2 changed files with 4 additions and 7 deletions.
diff --git a/.github/workflows/build.yml b/.github/workflows/build.yml
@@ -47,10 +47,7 @@ jobs:
     runs-on: ubuntu-latest
     steps:
       - uses: actions/checkout@v4
-
-      # Change to stable once https://issues.apache.org/jira/browse/INFRA-26428 resolves
-      - uses: dtolnay/rust-toolchain@nightly
-
+      - uses: astral-sh/setup-uv@v5
       - name: Generate license file
         run: uv run --no-project python ./dev/create_license.py
       - uses: actions/upload-artifact@v4
@@ -94,7 +91,7 @@ jobs:
         uses: astral-sh/setup-uv@v5
 
       - name: Build Python package
-        run: uv run --no-project maturin build --release --strip --features substrait
+        run: uv run --no-project maturin build --release --strip --features substrait -uv
 
       - name: List Windows wheels
         if: matrix.os == 'windows-latest'
@@ -148,7 +145,7 @@ jobs:
         uses: astral-sh/setup-uv@v5
 
       - name: Build Python package
-        run: uv run --no-project maturin build --release --strip --features substrait
+        run: uv run --no-project maturin build --release --strip --features substrait -uv
 
       - name: List Mac wheels
         run: find target/wheels/

diff --git a/.github/workflows/test.yaml b/.github/workflows/test.yaml
@@ -82,7 +82,7 @@ jobs:
           RUST_BACKTRACE: 1
         run: |
           uv sync --dev --no-install-package datafusion
-          uv run --no-project maturin develop
+          uv run --no-project maturin develop -uv
           uv run --no-project pytest -v .
 
       - name: FFI unit tests