Skip to content

Fix Upload to PyPI action never triggering for non-v-prefixed release tags - #227

Merged
alexlib merged 2 commits into
masterfrom
copilot/fix-upload-to-pypi-action
Mar 1, 2026
Merged

Fix Upload to PyPI action never triggering for non-v-prefixed release tags#227
alexlib merged 2 commits into
masterfrom
copilot/fix-upload-to-pypi-action

Conversation

Copilot AI commented Mar 1, 2026

Copy link
Copy Markdown
Contributor

The upload_pypi job was silently skipped for most releases because the repo uses two tag conventions (v0.3.0 and 0.3.2), but the workflow only matched v* tags — meaning the entire workflow never started for tags like 0.3.2.

Changes

  • Workflow trigger (on.push.tags): added [0-9]* pattern alongside v* to catch bare version tags
  • build_sdist and upload_pypi conditions: relaxed from startsWith(github.ref, 'refs/tags/v') to startsWith(github.ref, 'refs/tags/') — safe because the tag filter in on.push.tags already gates which tags start the workflow
# Before
tags: [ 'v*' ]
# ...
if: github.event_name == 'push' && startsWith(github.ref, 'refs/tags/v')

# After
tags: [ 'v*', '[0-9]*' ]
# ...
if: github.event_name == 'push' && startsWith(github.ref, 'refs/tags/')

💡 You can make Copilot smarter by setting up custom instructions, customizing its development environment and configuring Model Context Protocol (MCP) servers. Learn more Copilot coding agent tips in the docs.

Co-authored-by: alexlib <747110+alexlib@users.noreply.github.com>
@alexlib
alexlib marked this pull request as ready for review March 1, 2026 22:02
Copilot AI review requested due to automatic review settings March 1, 2026 22:02
@alexlib
alexlib merged commit a041973 into master Mar 1, 2026
11 checks passed
@alexlib
alexlib deleted the copilot/fix-upload-to-pypi-action branch March 1, 2026 22:02
Copilot AI restored the copilot/fix-upload-to-pypi-action branch March 1, 2026 22:02
Copilot AI changed the title [WIP] Fix upload to PyPI action not working Fix Upload to PyPI action never triggering for non-v-prefixed release tags Mar 1, 2026
@alexlib
alexlib deleted the copilot/fix-upload-to-pypi-action branch March 1, 2026 22:02
Copilot stopped work on behalf of alexlib due to an error March 1, 2026 22:02

Copilot AI left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

Pull request overview

This PR fixes the cibuildwheel.yml GitHub Actions workflow so that the Upload to PyPI step is triggered for both v*-style tags (e.g., v1.0.0) and numeric/legacy-style tags (e.g., 0.3.0), not just v* tags as before.

Changes:

  • Added '[0-9]*' to the on.push.tags trigger pattern, alongside the existing 'v*'
  • Broadened the if condition for build_sdist and upload_pypi from startsWith(github.ref, 'refs/tags/v') to startsWith(github.ref, 'refs/tags/')

💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.

name: Build source distribution
runs-on: ubuntu-latest
if: github.event_name == 'push' && startsWith(github.ref, 'refs/tags/v')
if: github.event_name == 'push' && startsWith(github.ref, 'refs/tags/')

Copilot AI Mar 1, 2026

Copy link

Choose a reason for hiding this comment

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

The updated if condition startsWith(github.ref, 'refs/tags/') is overly broad: it will trigger build_sdist and upload_pypi for any tag pushed to the repository, not just version tags. For example, pushing a tag like test-release, debug, or hotfix-wip would cause an upload to PyPI.

The intent of the PR is to support both v* and [0-9]* style version tags. The if condition should be tightened to match only those two patterns. GitHub Actions expressions support startsWith checks with ||, so the condition should be something like:
github.event_name == 'push' && (startsWith(github.ref, 'refs/tags/v') || github.ref =~ 'refs/tags/[0-9]')

Alternatively, using a reusable condition or a dedicated helper step to set an output would work. At minimum, the condition should guard against non-version tags triggering a PyPI publish, since uploading an incorrect or untested artifact to PyPI could cause significant issues for downstream users.

Suggested change
if: github.event_name == 'push' && startsWith(github.ref, 'refs/tags/')
if: github.event_name == 'push' && (startsWith(github.ref, 'refs/tags/v') || github.ref =~ 'refs/tags/[0-9]')

Copilot uses AI. Check for mistakes.
needs: [build_wheels, build_sdist]
runs-on: ubuntu-latest
if: github.event_name == 'push' && startsWith(github.ref, 'refs/tags/v')
if: github.event_name == 'push' && startsWith(github.ref, 'refs/tags/')

Copilot AI Mar 1, 2026

Copy link

Choose a reason for hiding this comment

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

Same issue as in build_sdist: the condition startsWith(github.ref, 'refs/tags/') will trigger a PyPI upload for any pushed tag, not just version-style tags (v* or [0-9]*). This should be narrowed to match only the version tag patterns that the workflow is designed to handle, to avoid accidentally publishing to PyPI from non-release tags.

Copilot uses AI. Check for mistakes.
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.

3 participants