Skip to content

Commit

Permalink
Update macOS CI (#386)
Browse files Browse the repository at this point in the history
  • Loading branch information
WillAyd authored Nov 4, 2024
1 parent d053fd2 commit f632897
Show file tree
Hide file tree
Showing 2 changed files with 6 additions and 8 deletions.
4 changes: 2 additions & 2 deletions .github/workflows/pypi_publish.yml
Original file line number Diff line number Diff line change
Expand Up @@ -14,15 +14,15 @@ jobs:
runs-on: ${{ matrix.os }}
strategy:
matrix:
os: [ubuntu-24.04, windows-2022, macos-12, macos-14]
os: [ubuntu-24.04, windows-2022, macos-13, macos-14]

steps:
- uses: actions/checkout@v4

- name: Build wheels
uses: pypa/[email protected]
env:
MACOSX_DEPLOYMENT_TARGET: "10.14"
MACOSX_DEPLOYMENT_TARGET: "11.0"

- uses: actions/[email protected]
with:
Expand Down
10 changes: 4 additions & 6 deletions .github/workflows/unit-test.yml
Original file line number Diff line number Diff line change
Expand Up @@ -41,7 +41,7 @@ jobs:
strategy:
fail-fast: false
matrix:
os: [ubuntu-24.04, windows-2022, macos-12, macos-14]
os: [ubuntu-24.04, windows-2022, macos-13, macos-14]

steps:
- uses: actions/checkout@v4
Expand All @@ -52,16 +52,15 @@ jobs:
- name: Build wheels for ${{ matrix.os }}
uses: pypa/[email protected]
env:
MACOSX_DEPLOYMENT_TARGET: "10.14"
CMAKE_APPLE_SILICON_PROCESSOR: ${{ matrix.os == 'macos-14' && 'arm64' || '' }}
MACOSX_DEPLOYMENT_TARGET: "11.0"

test_no_tableauhyperapi:
name: Test wheels without tableauhyperapi on ${{ matrix.os }}
runs-on: ${{ matrix.os }}
strategy:
fail-fast: false
matrix:
os: [ubuntu-24.04, windows-2022, macos-12, macos-14]
os: [ubuntu-24.04, windows-2022, macos-13, macos-14]

steps:
- uses: actions/checkout@v4
Expand All @@ -73,5 +72,4 @@ jobs:
uses: pypa/[email protected]
env:
CIBW_TEST_REQUIRES: pytest pytest-xdist[psutil] pandas>=2.0.0 polars narwhals
MACOSX_DEPLOYMENT_TARGET: "10.14"
CMAKE_APPLE_SILICON_PROCESSOR: ${{ matrix.os == 'macos-14' && 'arm64' || '' }}
MACOSX_DEPLOYMENT_TARGET: "11.0"

0 comments on commit f632897

Please sign in to comment.