Skip to content
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

Revert to token authentication for PyPI wheel uploads #1819

Merged
merged 1 commit into from
Jul 23, 2024

Conversation

nicholasjng
Copy link
Contributor

Until the PyPI account is recovered, it should be possible to upload wheels with the GitHub secrets that were previously used.

Changes the PyPI upload action sourcing to point to the v1 stable release branch, which receives rolling updates and is the canonical way of including the wheel publishing action.

Uploading will probably need another release, because setuptools_scm needs to produce a clean tag that the PyPI API allows as an upload.

Until the PyPI account is recovered, it should be possible to upload
wheels with the GitHub secrets that were previously used.

Changes the PyPI upload action sourcing to point to the v1 stable release
branch, which receives rolling updates and is the canonical way of
including the wheel publishing action.

Uploading will probably need another release, because setuptools_scm
needs to produce a clean tag that the PyPI API allows as an upload.
@nicholasjng
Copy link
Contributor Author

As a progress update: I am almost done with the new nanobind-bazel release, only the Windows stubgen implementation is giving me some trouble.

After that's done, the following google-benchmark PyPI releases will have stub files packaged with them, which allows for IDE completions, static type-checking, and an altogether better dev workflow for Python developers of GBM..

@dmah42 dmah42 merged commit df44bf7 into google:main Jul 23, 2024
80 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