Skip to content

Conversation

zklaus
Copy link
Collaborator

@zklaus zklaus commented Sep 30, 2025

In #6352, building of wheels for win-arm64 was added. This PR aligns the naming in the workflow more closely with the rest of the workflow file and adds the more modern python -m build variant to the previously used direct invocation of setup.py bdist_wheel as is done for other platforms.

Copy link

vercel bot commented Sep 30, 2025

@zklaus is attempting to deploy a commit to the Meta Open Source Team on Vercel.

A member of the Team first needs to authorize it.

@meta-cla meta-cla bot added the CLA Signed This label is managed by the Facebook bot. Authors need to sign the CLA before a PR can be reviewed. label Sep 30, 2025
@zklaus zklaus marked this pull request as ready for review October 1, 2025 10:03
@zklaus
Copy link
Collaborator Author

zklaus commented Oct 1, 2025

Also pinging @alinpahontu2912 as author of the previous PR 👋.

@zklaus zklaus requested review from atalman and seemethere October 1, 2025 10:05
Copy link
Contributor

@atalman atalman left a comment

Choose a reason for hiding this comment

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

Hi @zklaus please add unit test to cover this new approach

@zklaus zklaus force-pushed the mitigate-setup.py-use branch 5 times, most recently from 88ec5bb to 1d24fed Compare October 16, 2025 14:44
@zklaus zklaus force-pushed the mitigate-setup.py-use branch from 1d24fed to a015e74 Compare October 16, 2025 15:34
@zklaus
Copy link
Collaborator Author

zklaus commented Oct 16, 2025

@atalman, done. Do you think it's ok like this?

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

CLA Signed This label is managed by the Facebook bot. Authors need to sign the CLA before a PR can be reviewed.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants