Refine and test use of cibuildwheel

Author: mwiesenberger

.github/workflows/python_publish.yml

@@ -1,8 +1,9 @@
 name: Build/publish Python Package
 
 # Adopted from gh:pybind/scikit_build_example
+# Read more on https://cibuildwheel.pypa.io/en/stable
 
-on:
+on: [push, pull-request]
   workflow_dispatch:
   pull_request:
   push:
@@ -25,8 +26,6 @@ jobs:
     runs-on: ubuntu-latest
     steps:
     - uses: actions/checkout@v4
-      with:
-        submodules: true
 
     - name: Build SDist
       run: pipx run build --sdist
@@ -41,7 +40,7 @@ jobs:
 
 
   build_wheels:
-    name: Wheels on ${{ matrix.os }}
+    name: Build wheels on ${{ matrix.os }}
     runs-on: ${{ matrix.os }}
     strategy:
       fail-fast: false
@@ -50,15 +49,18 @@ jobs:
 
     steps:
     - uses: actions/checkout@v4
-      with:
-        submodules: true
 
     - uses: astral-sh/setup-uv@v4
 
-    - uses: pypa/[email protected]
+    - name: Build wheels
+      uses: pypa/[email protected]
       env:
         CIBW_ENABLE: cpython-prerelease
         CIBW_ARCHS_WINDOWS: auto ARM64
+      with:
+        package-dir: .
+        output-dir: wheelhouse
+        config-file: "{package}/pyproject.toml"
 
     - name: Verify clean directory
       run: git diff --exit-code

pyproject.toml

@@ -35,13 +35,15 @@ build-backend = "scikit_build_core.build"
 build.verbose = true # cmake --build . -v ## only visible if pip install . -v is used
 cmake.args = ["-DCMAKE_POLICY_VERSION_MINIMUM=3.5"] # Necessary for nlohmann_json with cmake>4.0
 
-# gh:pypa/cibuildwheel
 # Since we distribute binary wheels we need to build for multiple os using
-# cibuildwheel (copied from scikit_build_example
+# gh:pypa/cibuildwheel
+# You can local test using (requires docker)
+# uvx cibuildwheel
 [tool.cibuildwheel]
 build-frontend = "build[uv]"
 test-command = "pytest {project}/tests"
 test-extras = ["test"]
+skip = "*-musllinux*" # Skip musllinux (conflicts with libcudacxx)
 
 [tool.cibuildwheel.pyodide]
 build-frontend = {name = "build", args = ["--exports", "whole_archive"]}