Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

chore(deps): Bump actions/upload-artifact from 3 to 4 #83

Merged
merged 2 commits into from
Feb 19, 2024
Merged

chore(deps): Bump actions/upload-artifact from 3 to 4 #83

Changes from all commits
Commits
Show all changes
2 commits
Select commit Hold shift + click to select a range
27f57a9
chore(deps): Bump actions/upload-artifact from 3 to 4
dependabot[bot] Feb 19, 2024
fdc418c
Separate out individual wheel uploads
maresb Feb 19, 2024
File filter

Filter by extension

Filter by extension
Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
43 changes: 34 additions & 9 deletions .github/workflows/ci.yml
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -24,6 +24,8 @@ jobs:
strategy:
fail-fast: false
matrix:
# If you update the targets here, be sure to also update the
# download-artifact step in the release job!
target: [x86_64]
# pytensor is currently giving me issues on the other targets...
#target: [x86_64, aarch64, s390x, ppc64le]
Expand All @@ -42,9 +44,9 @@ jobs:
dnf install -y clang-libs clang || apt install llvm-dev libclang-dev clang

- name: Upload wheels
uses: actions/upload-artifact@v3
uses: actions/upload-artifact@v4
with:
name: wheels
name: linux-${{ matrix.target }}-wheels
path: dist
- name: pytest
if: ${{ startsWith(matrix.target, 'x86_64') }}
Expand Down Expand Up @@ -76,6 +78,8 @@ jobs:
strategy:
fail-fast: false
matrix:
# If you update the targets here, be sure to also update the
# download-artifact step in the release job!
target: [x64]
steps:
- uses: actions/checkout@v4
Expand All @@ -101,9 +105,9 @@ jobs:
args: --release --out dist --find-interpreter
sccache: 'true'
- name: Upload wheels
uses: actions/upload-artifact@v3
uses: actions/upload-artifact@v4
with:
name: wheels
name: windows-${{ matrix.target }}-wheels
path: dist
- name: pytest
if: ${{ !startsWith(matrix.target, 'aarch64') }}
Expand All @@ -120,6 +124,8 @@ jobs:
strategy:
fail-fast: false
matrix:
# If you update the targets here, be sure to also update the
# download-artifact step in the release job!
target: [x86_64, aarch64]
steps:
- uses: actions/checkout@v4
Expand All @@ -136,9 +142,9 @@ jobs:
args: --release --out dist --find-interpreter
sccache: 'true'
- name: Upload wheels
uses: actions/upload-artifact@v3
uses: actions/upload-artifact@v4
with:
name: wheels
name: macos-${{ matrix.target }}-wheels
path: dist
- name: pytest
if: ${{ !startsWith(matrix.target, 'aarch64') }}
Expand All @@ -160,9 +166,9 @@ jobs:
command: sdist
args: --out dist
- name: Upload sdist
uses: actions/upload-artifact@v3
uses: actions/upload-artifact@v4
with:
name: wheels
name: sdist
path: dist

release:
Expand All @@ -171,9 +177,28 @@ jobs:
if: "startsWith(github.ref, 'refs/tags/')"
needs: [linux, windows, macos, sdist]
steps:
# Combine all the wheels and sdists into a single directory
- name: Download artifacts
uses: actions/download-artifact@v4
with:
name: linux-x86_64-wheels
path: dist
- uses: actions/download-artifact@v4
with:
name: windows-x64-wheels
path: dist
- uses: actions/download-artifact@v4
with:
name: macos-x86_64-wheels
path: dist
- uses: actions/download-artifact@v4
with:
name: macos-aarch64-wheels
path: dist
- uses: actions/download-artifact@v4
with:
name: wheels
name: sdist
path: dist
- name: Publish to PyPI
uses: PyO3/maturin-action@v1
env:
Expand Down