Skip to content

Commit

Permalink
Bump actions/upload-artifact from 3 to 4 (#560)
Browse files Browse the repository at this point in the history
* Bump actions/upload-artifact from 3 to 4

Bumps [actions/upload-artifact](https://github.com/actions/upload-artifact) from 3 to 4.
- [Release notes](https://github.com/actions/upload-artifact/releases)
- [Commits](actions/upload-artifact@v3...v4)

---
updated-dependencies:
- dependency-name: actions/upload-artifact
  dependency-type: direct:production
  update-type: version-update:semver-major
...

Signed-off-by: dependabot[bot] <[email protected]>

* Migrate pypi to GHA artifacts v4

* Bump download-artifact to v4

* Eliminate undefined matrix.python-version variable

* Upload/download each platform separately

* Use pattern arg to download-artifact

---------

Signed-off-by: dependabot[bot] <[email protected]>
Co-authored-by: dependabot[bot] <49699333+dependabot[bot]@users.noreply.github.com>
Co-authored-by: Ben Mares <[email protected]>
  • Loading branch information
dependabot[bot] and maresb authored Jul 18, 2024
1 parent cac9feb commit ad27dc7
Show file tree
Hide file tree
Showing 2 changed files with 33 additions and 17 deletions.
41 changes: 28 additions & 13 deletions .github/workflows/pypi.yml
Original file line number Diff line number Diff line change
Expand Up @@ -30,12 +30,13 @@ jobs:
- name: Build SDist
run: pipx run build --sdist

- uses: actions/upload-artifact@v3
- uses: actions/upload-artifact@v4
with:
name: sdist
path: dist/*.tar.gz

build_wheels:
name: Build ${{ matrix.python-version }} wheels on ${{ matrix.platform }}
name: Build wheels for ${{ matrix.platform }}
runs-on: ${{ matrix.platform }}
strategy:
matrix:
Expand All @@ -51,19 +52,27 @@ jobs:
- name: Build wheels
uses: pypa/[email protected]

- uses: actions/upload-artifact@v3
- uses: actions/upload-artifact@v4
with:
name: wheels-${{ matrix.platform }}
path: ./wheelhouse/*.whl

check_dist:
name: Check dist
needs: [make_sdist,build_wheels]
runs-on: ubuntu-22.04
steps:
- uses: actions/download-artifact@v3
- uses: actions/download-artifact@v4
with:
name: artifact
name: sdist
path: dist

- uses: actions/download-artifact@v4
with:
pattern: wheels-*
path: dist
merge-multiple: true

- name: Check SDist
run: |
mkdir -p test-sdist
Expand All @@ -83,12 +92,18 @@ jobs:
runs-on: ubuntu-latest
if: github.event_name == 'release' && github.event.action == 'published'
steps:
- uses: actions/download-artifact@v3
with:
name: artifact
path: dist
- uses: actions/download-artifact@v4
with:
name: sdist
path: dist

- uses: pypa/[email protected]
with:
user: __token__
password: ${{ secrets.pypi_password }}
- uses: actions/download-artifact@v4
with:
pattern: wheels-*
path: dist
merge-multiple: true

- uses: pypa/[email protected]
with:
user: __token__
password: ${{ secrets.pypi_password }}
9 changes: 5 additions & 4 deletions .github/workflows/test.yml
Original file line number Diff line number Diff line change
Expand Up @@ -187,9 +187,9 @@ jobs:
FLOAT32: ${{ matrix.float32 }}

- name: Upload coverage file
uses: actions/upload-artifact@v3
uses: actions/upload-artifact@v4
with:
name: coverage
name: coverage-${{ steps.matrix-id.outputs.id }}
path: coverage/coverage-${{ steps.matrix-id.outputs.id }}.xml

benchmarks:
Expand Down Expand Up @@ -273,10 +273,11 @@ jobs:
python -m pip install -U coverage>=5.1 coveralls
- name: Download coverage file
uses: actions/download-artifact@v3
uses: actions/download-artifact@v4
with:
name: coverage
pattern: coverage-*
path: coverage
merge-multiple: true

- name: Upload coverage to Codecov
uses: codecov/codecov-action@v4
Expand Down

0 comments on commit ad27dc7

Please sign in to comment.