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

Attempt to force use of trusted publishing #70

Merged
merged 4 commits into from
Dec 19, 2024
Merged

Attempt to force use of trusted publishing #70

merged 4 commits into from
Dec 19, 2024

Conversation

tpgillam
Copy link
Owner

@tpgillam tpgillam commented Dec 19, 2024

Looking at the failure here, it seems like uv might not be trying to use it, even though it's configured on pypi.

https://github.com/tpgillam/mt2/actions/runs/12402987390/job/34626056980

Publishing 41 files https://upload.pypi.org/legacy/
Note: Neither credentials nor keyring are configured, and there was an error fetching the trusted publishing token. If you don't want to use trusted publishing, you can ignore this error, but you need to provide credentials.
Trusted publishing error: Environment variable ACTIONS_ID_TOKEN_REQUEST_TOKEN not set, is the `id-token: write` permission missing?
Uploading mt2-1.2.3-cp310-cp310-macosx_10_9_universal2.whl (70.3KiB)
error: Failed to publish `dist/mt2-1.2.3-cp310-cp310-macosx_10_9_universal2.whl` to https://upload.pypi.org/legacy/
  Caused by: Failed to send POST request
  Caused by: Missing credentials for https://upload.pypi.org/legacy/
Error: Process completed with exit code 2.

The "error fetching the trusted publishing token" might be indicative of another problem. Let the whack-a-mole commence...

@tpgillam tpgillam self-assigned this Dec 19, 2024
@tpgillam tpgillam requested a review from Rupt December 19, 2024 00:26
@tpgillam tpgillam marked this pull request as ready for review December 19, 2024 00:27
Copy link
Collaborator

@Rupt Rupt left a comment

Choose a reason for hiding this comment

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

++moo

@tpgillam tpgillam merged commit a2b68d5 into main Dec 19, 2024
38 checks passed
@tpgillam tpgillam deleted the tg/trust branch December 19, 2024 07:45
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