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

ci.yml: use OIDC connect to publish pypi packages #1029

Merged
merged 1 commit into from
Nov 26, 2023

Conversation

james-rms
Copy link
Collaborator

@james-rms james-rms commented Nov 24, 2023

Public-Facing Changes

None.

Description

I previously configured OIDC in pypi and test.pypi for all of our MCAP packages, see for example https://pypi.org/manage/project/mcap-protobuf-support/settings/publishing/

This PR switches CI over to using that auth method.

Once this is done, I can invalidate our API tokens, and we can remove the PYPI_TOKEN github secrets from this repo.

@james-rms james-rms merged commit 849974a into main Nov 26, 2023
24 checks passed
@james-rms james-rms deleted the jrms/mcap-pypi-oidc branch November 26, 2023 23:21
Copy link
Member

@jtbandes jtbandes left a comment

Choose a reason for hiding this comment

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

👍

pezy pushed a commit to pezy/mcap that referenced this pull request Jan 11, 2024
### Public-Facing Changes

None.

### Description
I previously configured OIDC in pypi and test.pypi for all of our MCAP
packages, see for example
https://pypi.org/manage/project/mcap-protobuf-support/settings/publishing/

This PR switches CI over to using that auth method.

Once this is done, I can invalidate our API tokens, and we can remove
the PYPI_TOKEN github secrets from this repo.
<!-- describe what has changed, and motivation behind those changes -->

<!-- Link relevant Github issues. Use `Fixes foxglove#1234` to auto-close the
issue after merging. -->
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Development

Successfully merging this pull request may close these issues.

3 participants