ci: add separate step for macos with venv

k4black · May 14, 2024 · 65ddee8 · 65ddee8
1 parent f13c7f9
commit 65ddee8
Show file tree

Hide file tree

Showing 2 changed files with 8 additions and 7 deletions.
diff --git a/.github/workflows/publish.yml b/.github/workflows/publish.yml
@@ -57,10 +57,8 @@ jobs:
     steps:
       - name: Build wheel
         run: |
-          python3 -m venv .venv
-          source .venv/bin/activate
-          python3 -m pip install --upgrade build wheel setuptools
-          python3 -m build --wheel --sdist --outdir ./dist --no-isolation
+          python3 -m pip install --upgrade build
+          python3 -m build --wheel --sdist --outdir ./dist
       - uses: pypa/gh-action-pypi-publish@release/v1
 
   sync-to-hf-hub:

diff --git a/.github/workflows/test.yml b/.github/workflows/test.yml
@@ -56,12 +56,15 @@ jobs:
     runs-on: ${{ matrix.os }}
     steps:
       - uses: actions/checkout@v4
-      - name: Build wheel
+      - name: Make venv for macos to cover "error externally-managed-environment"
+        if: startsWith(matrix.os, 'macos')
         run: |
           python3 -m venv .venv
           source .venv/bin/activate
-          python3 -m pip install --upgrade build wheel setuptools
-          python3 -m build --wheel --sdist --outdir ./dist --no-isolation
+      - name: Build wheel
+        run: |
+          python3 -m pip install --upgrade build
+          python3 -m build --wheel --sdist --outdir ./dist
       - name: Show dist files
         run: ls -lah ./dist
         shell: bash