Fix packagr publish (#525)

Fix packagr release
fennel-ai · Aug 6, 2024 · b166528 · b166528
1 parent 28199e6
commit b166528
Show file tree

Hide file tree

Showing 2 changed files with 19 additions and 9 deletions.
diff --git a/.github/workflows/packagr_publish.yml b/.github/workflows/packagr_publish.yml
@@ -67,17 +67,20 @@ jobs:
 
       # Build the package using Maturin
       - name: Build the package using Maturin
-        run: maturin build --release --manifest-path ./server/fennel_data_lib/Cargo.toml --out dist --no-sdist
-
-      # Install the built package
+        run: maturin build --release --features "extension-module" --manifest-path ./server/fennel_data_lib/Cargo.toml --out dist 
+     
+     # Install the built package
       - name: Install built package
-        run: pip install dist/*.whl
+        run: |
+          source .venv/bin/activate
+          python -m pip install dist/*.whl
+          python -c "import fennel_data_lib"
 
       # Run Python tests
       - name: Run tests
         run: |
           source .venv/bin/activate
-          poetry run pytest fennel --run_slow
+          pytest fennel --run_slow
 
       # Publish the package to PyPI if all tests pass
       - name: Publish to PyPI

diff --git a/.github/workflows/test_linux.yml b/.github/workflows/test_linux.yml
@@ -89,9 +89,16 @@ jobs:
       - name: Install maturin
         run: python -m pip install maturin
 
-      - name: Build and link the package using Maturin Develop
-        run: maturin develop --release --manifest-path ./server/fennel_data_lib/Cargo.toml
+      # Build the package using Maturin
+      - name: Build the package using Maturin
+        run: maturin build --release --features "extension-module" --manifest-path ./server/fennel_data_lib/Cargo.toml --out dist 
 
+      # Install the built package
+      - name: Install built package
+        run: |
+          source .venv/bin/activate
+          python -m pip install dist/*.whl
+          python -c "import fennel_data_lib"
 
       #----------------------------------------------
       # install your root project, if required
@@ -103,6 +110,6 @@ jobs:
       - name: Run tests
         run: |
           source .venv/bin/activate
-          poetry run pytest fennel --run_slow
-          poetry run pytest docs
+          pytest fennel --run_slow
+          pytest docs
           poetry run python -m unittest discover fennel