Skip to content

Update wheel build dependencies #21

Merged
niemilau merged 7 commits intomainfrom
update-deps
Dec 12, 2025
Merged

Update wheel build dependencies #21
niemilau merged 7 commits intomainfrom
update-deps

Conversation

@niemilau
Copy link
Contributor

@niemilau niemilau commented Dec 12, 2025

Now builds wheels for Python 3.14 too.

  • Skip PyPy wheels as they now fail to build due to some C-extension incompatibilities. Quite miraculous that they worked until now.
    -MacOS-x86 still gets wrong arch somehow and fails so not including that.

https://github.com/Wall-Go/WallGoCollision/actions/runs/20160749787

Copy link

Copilot AI left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Pull request overview

This PR updates the wheel build infrastructure to support Python 3.14 and addresses build compatibility issues. The changes modernize the build tooling by upgrading cibuildwheel to v3.3.0, exclude problematic PyPy wheels due to C-extension incompatibilities, and update the muparser dependency to version 2.3.5. The PR also includes minor code cleanup with trailing whitespace removal.

  • Upgraded cibuildwheel from v2.21.3 to v3.3.0 to enable Python 3.14 wheel builds
  • Added PyPy wheels to skip list to avoid C-extension build failures
  • Updated muparser dependency from 2.3.4 to 2.3.5

Reviewed changes

Copilot reviewed 3 out of 3 changed files in this pull request and generated no comments.

File Description
pyproject.toml Updated skip configuration to exclude PyPy wheels in addition to musllinux, with improved comment documentation
conanfile.py Bumped muparser dependency from 2.3.4 to 2.3.5 and removed trailing whitespace
.github/workflows/wheels.yml Upgraded cibuildwheel action from v2.21.3 to v3.3.0, updated macOS runner comment, and cleaned up trailing whitespace

💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.

@jorindevandevis jorindevandevis self-requested a review December 12, 2025 10:09
Copy link
Contributor

@jorindevandevis jorindevandevis left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Since it works, and copilot does not have comments, I am happy to approve this PR.

@niemilau niemilau merged commit 0621f65 into main Dec 12, 2025
8 checks passed
@niemilau niemilau deleted the update-deps branch December 12, 2025 11:32
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants