Skip to content

Move most cibuildwheel options to pyproject.toml #748

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.

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

Open
wants to merge 4 commits into
base: master
Choose a base branch
from

Conversation

mhucka
Copy link
Collaborator

@mhucka mhucka commented May 30, 2025

This adds a pyproject.toml file to the top level of the repository. The initial contents are settings for cibuildwheel and pytest. The cibuildwheel settings came from the GitHub Actions workflow for building wheels. Putting the settings here means the settings are available in more situations, such as when running cibuildwheel manually for testing, or if we create more scripts or workflows that run cibuildwheel. Since command-line options and environment variables override cibuildwheel values in pyproject.toml, these additions should not affect existing scripts and workflows.

The purpose of doing this now is to prepare for revised GitHub Actions workflows coming in another PR. These changes here need to be in place before the workflow changes.

Note: CI checks are failing, but not due to this file; it's because the CI workflows and other changes need to be made.

The initial contents are settings for cibuildwheel and pytest. The
cibuildwheel settings came from the GitHub Actions workflow for
building wheels. Putting the settings here means the settings are
available in more situations, such as when running cibuildwheel
manually for testing, or if we create more scripts or workflows that
run cibuildwheel.
@mhucka mhucka self-assigned this May 30, 2025
@mhucka mhucka requested review from pavoljuhas and dstrain115 May 30, 2025 18:11
@mhucka mhucka changed the title Start a pyproject.toml file Move most cibuildwheel options to pyproject.toml May 31, 2025
@mhucka mhucka marked this pull request as draft May 31, 2025 20:01
@mhucka mhucka marked this pull request as ready for review June 10, 2025 03:27
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.

1 participant