Skip to content

Update PyPI publishing workflow #834

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

Draft
wants to merge 9 commits into
base: master
Choose a base branch
from
Draft

Conversation

mhucka
Copy link
Collaborator

@mhucka mhucka commented Jul 8, 2025

This updates the workflows for publishing wheels. The main changes are:

  • Add additional tests after the cibuildwheel step in reusable_build_wheels.yaml. These check the structure of the wheels in ways that cibuildwheel does not.
  • Add support for manually publishing to TestPyPI
  • When debug is true, use verbose flag to publishing action
  • Set permissions on the upload job in the workflow
  • Update job names and timeouts

mhucka added 6 commits July 8, 2025 16:09
cibuildwheel seems to set a thread number automatically. My previous
code to set threads is unnecessary and adds complexity.
It is unlikely that this would be triggered multiple times, but given
that it's possible to run this workflow manually, it's best to guard
against accidental concurrent execution.
`cibuildwheel` does not run `twine check`. It makes sense to run this
quick check as part of building & testing wheels.

[`check-wheel-contents`](https://github.com/jwodder/check-wheel-contents)
is a utility that performs other checks on the wheel structure, such as
whether there are extraneous files or directories in the wheel.
- Add support for manually publishing to TestPyPI
- When debug is true, use verbose flag to publishing action
- Set permissions on upload job
- Update job names and timeouts
@mhucka mhucka changed the title Remove setting of no. of threads Update PyPI publishing workflow Jul 8, 2025
MacOS and Linux top-level structures are different, and at the moment I
don't see a way of setting alternate values for different platforms.
@mhucka mhucka force-pushed the mh-update-pypi-workflow branch from 1de4e87 to 04d3dea Compare July 8, 2025 20:02
@mhucka mhucka self-assigned this Aug 1, 2025
@mhucka mhucka added the area/devops Involves build systems, Make files, Bazel files, continuous integration, and or other DevOps topics label Aug 13, 2025
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
area/devops Involves build systems, Make files, Bazel files, continuous integration, and or other DevOps topics size: M
Projects
None yet
Development

Successfully merging this pull request may close these issues.

1 participant