Skip to content

Move static cibuildwheel config into pyproject.toml#1

Merged
jspanchu merged 3 commits into
wheelfrom
refactor-cibw-for-local-builds
Jun 12, 2026
Merged

Move static cibuildwheel config into pyproject.toml#1
jspanchu merged 3 commits into
wheelfrom
refactor-cibw-for-local-builds

Conversation

@jspanchu

Copy link
Copy Markdown
Member

Relocate the CIBW environment variables (SDK index, GL build deps, retag-only repair command, test command) from build-wheels.yml into [tool.cibuildwheel] so local builds and CI share the same settings; CI now only selects the build target. Document local wheel builds in the README.

@jspanchu jspanchu requested a review from AlexyPellegrini June 12, 2026 14:41

@AlexyPellegrini AlexyPellegrini left a comment

Copy link
Copy Markdown
Collaborator

Choose a reason for hiding this comment

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

LGTM pending comments

Comment thread README.md Outdated
Comment thread README.md Outdated
Comment thread README.md Outdated
Comment thread README.md Outdated
@jspanchu jspanchu force-pushed the refactor-cibw-for-local-builds branch from 3bb5cdc to 741d7df Compare June 12, 2026 15:06
jspanchu added 2 commits June 12, 2026 11:08
Relocate the CIBW environment variables (SDK index, GL build deps,
retag-only repair command, test command) from build-wheels.yml into
[tool.cibuildwheel] so local builds and CI share the same settings;
CI now only selects the build target. Document local wheel builds
in the README.
@jspanchu jspanchu force-pushed the refactor-cibw-for-local-builds branch from 741d7df to 85a0ccf Compare June 12, 2026 15:08

@AlexyPellegrini AlexyPellegrini left a comment

Copy link
Copy Markdown
Collaborator

Choose a reason for hiding this comment

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

Thanks!

@jspanchu

Copy link
Copy Markdown
Member Author

I've also added steps to build using pip install .. for someone that does not want to use cibw.

@jspanchu jspanchu merged commit dd826af into wheel Jun 12, 2026
3 checks passed
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