Skip to content

Commit 87f478f

Browse files
committed
EBR-75. Update wheels workflow to fix breaking changes in actions/upload&download@v4 and temporarily disable release on pypi
1 parent 4052646 commit 87f478f

File tree

1 file changed

+29
-16
lines changed

1 file changed

+29
-16
lines changed

.github/workflows/wheels.yml

Lines changed: 29 additions & 16 deletions
Original file line numberDiff line numberDiff line change
@@ -1,8 +1,6 @@
11
name: Build
22

3-
on:
4-
release:
5-
types: [published]
3+
on: push
64

75
jobs:
86
build_wheels:
@@ -30,6 +28,7 @@ jobs:
3028

3129
- uses: actions/upload-artifact@v4
3230
with:
31+
name: wheels-${{ matrix.os }}
3332
path: ./wheelhouse/*.whl
3433

3534
build_sdist:
@@ -43,24 +42,38 @@ jobs:
4342

4443
- uses: actions/upload-artifact@v4
4544
with:
45+
name: sdist
4646
path: dist/*.tar.gz
4747

48-
upload_pypi:
48+
merge_artifacts:
49+
name: Merge artifacts
4950
needs: [build_wheels, build_sdist]
5051
runs-on: ubuntu-latest
5152
steps:
52-
- uses: actions/download-artifact@v4
53+
- name: Merge wheels
54+
uses: actions/upload-artifact/merge@v4
5355
with:
54-
# unpacks default artifact into dist/
55-
# if `name: artifact` is omitted, the action will create extra parent dir
56-
name: artifact
56+
name: merged-artifact
57+
patterns: |
58+
wheels-*
59+
sdist
60+
61+
upload_pypi:
62+
name: Upload to PyPI
63+
needs: merge_artifacts
64+
runs-on: ubuntu-latest
65+
steps:
66+
- name: Download merged artifact
67+
uses: actions/download-artifact@v4
68+
with:
69+
name: merged-artifact
5770
path: dist
5871

59-
- name: Release on Pypi
60-
env:
61-
TWINE_USERNAME: ${{ secrets.PYPI_USER }}
62-
TWINE_PASSWORD: ${{ secrets.PYPI_PASS }}
63-
run: |
64-
echo "Publishing on pypi..."
65-
python -m pip install -U twine
66-
twine upload dist/*
72+
# - name: Release on Pypi
73+
# env:
74+
# TWINE_USERNAME: ${{ secrets.PYPI_USER }}
75+
# TWINE_PASSWORD: ${{ secrets.PYPI_PASS }}
76+
# run: |
77+
# echo "Publishing on pypi..."
78+
# python -m pip install -U twine
79+
# twine upload dist/*

0 commit comments

Comments
 (0)