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

Conversation

@hamishfagg
Copy link
Contributor

No description provided.

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.

3 participants