Skip to content

Commit c88f015

Browse files
committed
workflows: add CPU architecture to names of workflow artifacts
We'll need to ensure uniqueness of Linux workflow artifact names once we're also building for aarch64, so make the change on all OSes for consistency. The change doesn't affect the filenames inside the artifacts, so shouldn't be user-visible. Signed-off-by: Benjamin Gilbert <[email protected]>
1 parent 6d469ff commit c88f015

File tree

1 file changed

+4
-4
lines changed

1 file changed

+4
-4
lines changed

.github/workflows/python.yml

Lines changed: 4 additions & 4 deletions
Original file line numberDiff line numberDiff line change
@@ -93,11 +93,11 @@ jobs:
9393
run: |
9494
case "${{ matrix.os }}" in
9595
ubuntu-latest)
96-
echo OS_TAG=linux >> $GITHUB_ENV
96+
echo OS_ARCH_TAG=linux-x86_64 >> $GITHUB_ENV
9797
sudo apt-get install libopenslide0
9898
;;
9999
macos-latest)
100-
echo OS_TAG=macos >> $GITHUB_ENV
100+
echo OS_ARCH_TAG=macos-arm64-x86_64 >> $GITHUB_ENV
101101
echo DYLD_LIBRARY_PATH=/opt/homebrew/lib >> $GITHUB_ENV
102102
brew install openslide
103103
;;
@@ -151,7 +151,7 @@ jobs:
151151
if: env.archive_wheel
152152
uses: actions/upload-artifact@v4
153153
with:
154-
name: ${{ needs.pre-commit.outputs.dist-base }}-${{ env.OS_TAG }}-${{ matrix.python-version }}
154+
name: ${{ needs.pre-commit.outputs.dist-base }}-${{ env.OS_ARCH_TAG }}-${{ matrix.python-version }}
155155
path: artifacts/whl
156156
compression-level: 0
157157

@@ -223,7 +223,7 @@ jobs:
223223
if: env.archive_wheel
224224
uses: actions/upload-artifact@v4
225225
with:
226-
name: ${{ needs.pre-commit.outputs.dist-base }}-windows-${{ matrix.python-version }}
226+
name: ${{ needs.pre-commit.outputs.dist-base }}-windows-x64-${{ matrix.python-version }}
227227
path: artifacts/whl
228228
compression-level: 0
229229

0 commit comments

Comments
 (0)