Skip to content

Fix missing tag trigger for PyPI release workflow#291

Open
Copilot wants to merge 2 commits intodevelfrom
copilot/fix-tags-command-in-yaml
Open

Fix missing tag trigger for PyPI release workflow#291
Copilot wants to merge 2 commits intodevelfrom
copilot/fix-tags-command-in-yaml

Conversation

Copy link
Copy Markdown

Copilot AI commented Mar 30, 2026

The release_pypi job's if: condition checks startsWith(github.ref, 'refs/tags/'), but the workflow's on.push only listed branches — so pushing a tag never triggered the workflow in tag context. github.ref remained refs/heads/devel, making the condition permanently false.

Change

Added tags: ["v*"] to the push trigger:

on:
  push:
    branches: [ "devel", "main" ]
    tags:
      - "v*"

This ensures pushing a tag like v2.1.0 triggers the workflow with github.ref = refs/tags/v2.1.0, allowing the buildrelease_pypi chain to execute correctly.


🔒 GitHub Advanced Security automatically protects Copilot coding agent pull requests. You can protect all pull requests by enabling Advanced Security for your repositories. Learn more about Advanced Security.

Copilot AI linked an issue Mar 30, 2026 that may be closed by this pull request
Agent-Logs-Url: https://github.com/openghg/fluxie/sessions/6a30c5b1-232b-47b4-9655-7623a1d52745

Co-authored-by: lionel42 <43442120+lionel42@users.noreply.github.com>
Copilot AI changed the title [WIP] Fix missing tags command in CI YAML for versioning Fix missing tag trigger for PyPI release workflow Mar 30, 2026
Copilot AI requested a review from lionel42 March 30, 2026 08:44
@lionel42
Copy link
Copy Markdown
Collaborator

lionel42 commented Mar 30, 2026

Are you sure this is going to work when I tag ? Maybe link to the official doc ? @copilot

@lionel42 lionel42 marked this pull request as ready for review March 30, 2026 09:16
@lionel42 lionel42 requested a review from SutarPrasad March 30, 2026 09:22
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.

Versioning

2 participants