Skip to content
This repository was archived by the owner on May 14, 2025. It is now read-only.

fix token name #85

Merged
merged 1 commit into from
May 6, 2025
Merged

fix token name #85

merged 1 commit into from
May 6, 2025

Conversation

hamishfagg
Copy link
Contributor

No description provided.

@@ -83,7 +83,7 @@ jobs:
- name: Build and publish
env:
TWINE_USERNAME: __token__
TWINE_PASSWORD: ${{ secrets.PYPI_TOKEN }}
TWINE_PASSWORD: ${{ secrets.PYPI_PASSWORD }}

Choose a reason for hiding this comment

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

Correctness: The GitHub Actions workflow is referencing secrets.PYPI_PASSWORD instead of secrets.PYPI_TOKEN, which may cause authentication failures when publishing to PyPI if the secret is incorrectly named in the repository settings.

📝 Committable Code Suggestion

‼️ Ensure you review the code suggestion before committing it to the branch. Make sure it replaces the highlighted code, contains no missing lines, and has no issues with indentation.

Suggested change
TWINE_PASSWORD: ${{ secrets.PYPI_PASSWORD }}
TWINE_PASSWORD: ${{ secrets.PYPI_TOKEN }}

@hamishfagg hamishfagg merged commit 14c46fe into staging May 6, 2025
2 of 4 checks passed
@hamishfagg hamishfagg deleted the fix/token_name branch May 6, 2025 03:11
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants