Skip to content

Commit

Permalink
Do not upload to the same artifact twice
Browse files Browse the repository at this point in the history
This is not possible anymore with upload-artifact@v4.
  • Loading branch information
coldfix committed Apr 22, 2024
1 parent fdb4d75 commit 83f300a
Showing 1 changed file with 7 additions and 7 deletions.
14 changes: 7 additions & 7 deletions .github/workflows/build.yml
Original file line number Diff line number Diff line change
Expand Up @@ -80,7 +80,7 @@ jobs:
- name: Upload cpymad wheels
uses: actions/upload-artifact@v4
with:
name: dist-linux
name: dist-linux-${{ matrix.base }}
path: dist

build_windows:
Expand Down Expand Up @@ -261,19 +261,19 @@ jobs:
- name: Upload cpymad wheels
uses: actions/upload-artifact@v4
with:
name: dist-macos
name: dist-macos-${{ matrix.arch }}
path: dist

test:
name: Tests
runs-on: ${{ matrix.os == 'linux' && 'ubuntu' || matrix.os }}-latest
runs-on: ${{ matrix.os.run }}
needs: [build_linux, build_windows, build_macos]
strategy:
matrix:
os:
- linux
- windows
- macos
- {run: ubuntu-latest, dist: dist-linux-manylinux2014_x86_64}
- {run: windows-latest, dist: dist-windows}
- {run: macos-latest, dist: dist-macos-x86_64}
python:
- '3.7'
- '3.8'
Expand All @@ -295,7 +295,7 @@ jobs:
- name: Download cpymad wheels
uses: actions/download-artifact@v4
with:
name: dist-${{ matrix.os }}
name: ${{ matrix.os.dist }}
path: dist

- name: Install cpymad from wheel
Expand Down

0 comments on commit 83f300a

Please sign in to comment.