Skip to content

Commit

Permalink
Update workflow to use upload-artifact v4 and download-artifact v4
Browse files Browse the repository at this point in the history
This updates the CI/CD workflow to use the `upload-artifact` v4 and
`download-artifact` v4 GitHub Actions. The currently used `upload-artifact`
and `download-artifact` will no longer work at the end of next month.
The changes are needed since v4 no longer has mutable artifacts, which
was used to collect wheels from different architectures.

Fixes #8588, fixes #8589, fixes #9009, fixes #10189, fixes #10191.

PR #10281.
  • Loading branch information
silamon authored Jan 1, 2025
1 parent e5d6016 commit d54b3e2
Show file tree
Hide file tree
Showing 2 changed files with 18 additions and 11 deletions.
28 changes: 17 additions & 11 deletions .github/workflows/ci-cd.yml
Original file line number Diff line number Diff line change
Expand Up @@ -127,7 +127,7 @@ jobs:
run: |
make generate-llhttp
- name: Upload llhttp generated files
uses: actions/upload-artifact@v3
uses: actions/upload-artifact@v4
with:
name: llhttp
path: vendor/llhttp/build
Expand Down Expand Up @@ -193,7 +193,7 @@ jobs:
python -m pip install -r requirements/test.in -c requirements/test.txt
- name: Restore llhttp generated files
if: ${{ matrix.no-extensions == '' }}
uses: actions/download-artifact@v3
uses: actions/download-artifact@v4
with:
name: llhttp
path: vendor/llhttp/build/
Expand Down Expand Up @@ -280,7 +280,7 @@ jobs:
run: |
python -m pip install -r requirements/test.in -c requirements/test.txt
- name: Restore llhttp generated files
uses: actions/download-artifact@v3
uses: actions/download-artifact@v4
with:
name: llhttp
path: vendor/llhttp/build/
Expand Down Expand Up @@ -344,7 +344,7 @@ jobs:
python -m
pip install -r requirements/cython.in -c requirements/cython.txt
- name: Restore llhttp generated files
uses: actions/download-artifact@v3
uses: actions/download-artifact@v4
with:
name: llhttp
path: vendor/llhttp/build/
Expand All @@ -355,9 +355,9 @@ jobs:
run: |
python -m build --sdist
- name: Upload artifacts
uses: actions/upload-artifact@v3
uses: actions/upload-artifact@v4
with:
name: dist
name: dist-sdist
path: dist

build-wheels:
Expand Down Expand Up @@ -410,7 +410,7 @@ jobs:
python -m
pip install -r requirements/cython.in -c requirements/cython.txt
- name: Restore llhttp generated files
uses: actions/download-artifact@v3
uses: actions/download-artifact@v4
with:
name: llhttp
path: vendor/llhttp/build/
Expand All @@ -421,9 +421,15 @@ jobs:
uses: pypa/[email protected]
env:
CIBW_ARCHS_MACOS: x86_64 arm64 universal2
- uses: actions/upload-artifact@v3
- name: Upload wheels
uses: actions/upload-artifact@v4
with:
name: dist
name: >-
dist-${{ matrix.os }}-${{
matrix.qemu
&& matrix.qemu
|| 'native'
}}
path: ./wheelhouse/*.whl

deploy:
Expand All @@ -448,10 +454,10 @@ jobs:
run: |
echo "${{ secrets.GITHUB_TOKEN }}" | gh auth login --with-token
- name: Download distributions
uses: actions/download-artifact@v3
uses: actions/download-artifact@v4
with:
name: dist
path: dist
pattern: dist-*
- name: Collected dists
run: |
tree dist
Expand Down
1 change: 1 addition & 0 deletions CHANGES/10281.contrib.rst
Original file line number Diff line number Diff line change
@@ -0,0 +1 @@
The CI/CD workflow has been updated to use `upload-artifact` v4 and `download-artifact` v4 GitHub Actions -- by :user:`silamon`.

0 comments on commit d54b3e2

Please sign in to comment.