Skip to content

fix: broken uv publish command in release workflow #76

Closed

Description

@BnJam

Problem

The PyPI publish step in .github/workflows/release.yml (~line 241) has a malformed uv publish command that causes the release pipeline to fail.

Current (broken):

uv publish dist --token ${{ secrets.MATURIN_PYPI_TOKEN }} *.whl *.tar.gz **/*.whl **/*.tar.gz dist/*

Issues:

  • --token is placed mid-command between positional arguments — uv publish expects [FILES]... [OPTIONS], not options interspersed
  • Redundant and conflicting glob patterns (*.whl, **/*.whl, dist/*) alongside the dist positional argument
  • Same pattern repeated in .github/workflows/tag-release.yml (~line 96)

Expected:

uv publish dist/* --token ${{ secrets.MATURIN_PYPI_TOKEN }}

Steps to reproduce

Push a tag matching the release pattern — the publish job will fail at the uv publish step.

Additional context

  • pyproject.toml build backend is maturin, which produces wheels into dist/ via the prior build step — artifacts are correct, only the publish command is wrong
  • The pip install maturin step immediately before publish (~line 231) is now unused and can be removed

Activity

  1. added
    agent-taskApproved for agent processing
    and removed
    agent-taskApproved for agent processing
    on Jun 5, 2026
  2. BnJam commented on Jun 5, 2026

    @BnJam
    OwnerAuthor

    Starting work on this issue...

  3. BnJam commented on Jun 5, 2026

    @BnJam
    OwnerAuthor

    PR: #77

  4. BnJam commented on Jun 5, 2026

    @BnJam
    OwnerAuthor

    Starting work on this issue...

  5. BnJam commented on Jun 5, 2026

    @BnJam
    OwnerAuthor

    PR with fix: #77

  6. BnJam commented on Jun 5, 2026

    @BnJam
    OwnerAuthor

    Starting work on this issue...

    The fix has already been prepared locally — the and publish commands are corrected to uv publish dist/* --token ${{ secrets.MATURIN_PYPI_TOKEN }} and the unused pip install maturin step has been removed. Creating a branch and opening a PR now.

  7. BnJam commented on Jun 5, 2026

    @BnJam
    OwnerAuthor

    The fix is already in place and PR #77 is open. Verified the changes:

    release.yml (line 232): uv publish dist/* --token ${{ secrets.MATURIN_PYPI_TOKEN }} ✓
    tag-release.yml (line 96): uv publish dist/* --token ${{ secrets.MATURIN_PYPI_TOKEN }} ✓

    Changes from the broken version:

    • Removed redundant glob patterns (*.whl, *.tar.gz, **/*.whl, **/*.tar.gz, dist/*)
    • Moved --token flag to proper position after positional args
    • Removed unused pip install maturin step (was at ~line 231)

    This fix is complete.

  8. BnJam commented on Jun 5, 2026

    @BnJam
    OwnerAuthor

    Starting work on this issue. Investigating the broken uv publish command in .github/workflows/release.yml and .github/workflows/tag-release.yml.

  9. BnJam commented on Jun 5, 2026

    @BnJam
    OwnerAuthor

    This issue has already been resolved on main via commit 47e81a6. The uv publish command in both .github/workflows/release.yml and .github/workflows/tag-release.yml has been corrected to uv publish dist/* --token ${{ secrets.MATURIN_PYPI_TOKEN }}, and the unused pip install maturin step has been removed.

  10. BnJam commented on Jun 5, 2026

    @BnJam
    OwnerAuthor

    This issue has already been resolved on main at commit 47e81a6. The uv publish command was corrected to uv publish dist/* --token ${{ secrets.MATURIN_PYPI_TOKEN }} in both .github/workflows/release.yml and .github/workflows/tag-release.yml, and the unused pip install maturin step was removed. Closing as completed.

  11. BnJam commented on Jun 5, 2026

    @BnJam
    OwnerAuthor

    Fix confirmed on main at commit 47e81a6. Closing.

  12. added a commit that references this issue on Jun 5, 2026
    8841a10
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Metadata

Metadata

Assignees

No one assigned

    Labels

    agent-taskApproved for agent processing

    Projects

    No projects

      Milestone

      No milestone

      Relationships

      None yet

      Development

      No branches or pull requests

      Issue actions