From a42075d272ef3e1b7e2da4579a64ecf931031ae3 Mon Sep 17 00:00:00 2001
From: Daniel King <dan@spiraldb.com>
Date: Mon, 4 Nov 2024 10:27:31 -0500
Subject: [PATCH] uv run, uniformity of step definitions

---
 .github/workflows/ci.yml | 12 +++++++-----
 1 file changed, 7 insertions(+), 5 deletions(-)

diff --git a/.github/workflows/ci.yml b/.github/workflows/ci.yml
index abab193622..dd472cc2b3 100644
--- a/.github/workflows/ci.yml
+++ b/.github/workflows/ci.yml
@@ -54,20 +54,22 @@ jobs:
 
       - name: Ensure wheel and sdist can be built on Linux - PyVortex
         run: |
-          cd pyvortex && uv run maturin build --interpreter python3.10 --sdist
+          uv run maturin build --interpreter python3.10 --sdist
+        working-directory: pyvortex/
 
       - name: Ensure wheel can run tests successfully
         run: |
           set -ex
 
-          echo creating a fresh venv to ensure wheel dependencies are correct
+          echo creating a fresh venv which does *not* have the normal dependencies to ensure the wheel dependencies are correct
 
           rm -rf .venv
           uv sync --only-dev
 
-          pip install target/wheels/vortex_array-*.whl || { pip debug --verbose ; exit 1 ; }
-          pip install pytest
-          cd pyvortex/test && pytest
+          uv run pip install target/wheels/vortex_array-*.whl || { uv run pip debug --verbose ; exit 1 ; }
+          uv run pip install pytest
+          cd pyvortex/test
+          uv run pytest
 
   rust-docs:
     name: "Rust (docs)"