Skip to content

Commit

Permalink
ci: use upload-artifact@v4 for release publishing
Browse files Browse the repository at this point in the history
We can no longer upload to the same named artifact across
multiple jobs. So we need to incorporate the build optimization level
into the artifact name.

I think release automation will continue to work since we iterate
over every artifact, download, and inspect the files within. But there
is a chance we need to tweak release automation to account for this
change.
  • Loading branch information
indygreg committed Feb 19, 2024
1 parent 17099a3 commit 3a28398
Show file tree
Hide file tree
Showing 3 changed files with 6 additions and 6 deletions.
4 changes: 2 additions & 2 deletions .github/workflows/apple.yml
Original file line number Diff line number Diff line change
Expand Up @@ -195,9 +195,9 @@ jobs:
./build-macos.py --target-triple ${{ matrix.build.target_triple }} --python ${{ matrix.build.py }} --optimizations ${{ matrix.build.optimizations }}
- name: Upload Distributions
uses: actions/upload-artifact@v3
uses: actions/upload-artifact@v4
with:
name: ${{ matrix.build.py }}-${{ matrix.build.target_triple }}
name: ${{ matrix.build.py }}-${{ matrix.build.target_triple }}-${{ matrix.build.optimizations }}
path: dist/*

- uses: actions/checkout@v4
Expand Down
4 changes: 2 additions & 2 deletions .github/workflows/linux.yml
Original file line number Diff line number Diff line change
Expand Up @@ -947,7 +947,7 @@ jobs:
build/pythonbuild validate-distribution ${EXTRA_ARGS} dist/*.tar.zst
- name: Upload Distribution
uses: actions/upload-artifact@v3
uses: actions/upload-artifact@v4
with:
name: ${{ matrix.build.py }}-${{ matrix.build.target_triple }}
name: ${{ matrix.build.py }}-${{ matrix.build.target_triple }}-${{ matrix.build.optimizations }}
path: dist/*
4 changes: 2 additions & 2 deletions .github/workflows/windows.yml
Original file line number Diff line number Diff line change
Expand Up @@ -92,7 +92,7 @@ jobs:
.\pythonbuild.exe validate-distribution --run $Dists
- name: Upload Distributions
uses: actions/upload-artifact@v3
uses: actions/upload-artifact@v4
with:
name: ${{ matrix.py }}-${{ matrix.vcvars }}
name: ${{ matrix.py }}-${{ matrix.vcvars }}-${{ matrix.profile }}
path: dist/*

0 comments on commit 3a28398

Please sign in to comment.