Skip to content

Fix tag computation in final version check.#174

Merged
sebschmi merged 3 commits intomainfrom
fix_python_release
May 7, 2026
Merged

Fix tag computation in final version check.#174
sebschmi merged 3 commits intomainfrom
fix_python_release

Conversation

@sebschmi
Copy link
Copy Markdown
Collaborator

@sebschmi sebschmi commented May 7, 2026

No description provided.

Copilot AI review requested due to automatic review settings May 7, 2026 11:46
@sebschmi sebschmi enabled auto-merge May 7, 2026 11:46
Copy link
Copy Markdown
Contributor

Copilot AI left a comment

Choose a reason for hiding this comment

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

Pull request overview

Updates the GitHub Actions workflow’s “Check Versions” step to compute the release tag based on fetched Git tags rather than relying on GITHUB_REF, aiming to ensure the version check uses the intended release tag format (py_lib_tsalign-v*).

Changes:

  • Fetches tags in the release job before computing the tag used for version comparison.
  • Computes the latest py_lib_tsalign-v* tag via git tag -l ... | sort -V | tail -n 1.

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

Comment thread .github/workflows/python.yml Outdated
sebschmi and others added 2 commits May 7, 2026 14:48
Co-authored-by: Copilot Autofix powered by AI <175728472+Copilot@users.noreply.github.com>
Copy link
Copy Markdown
Contributor

Copilot AI left a comment

Choose a reason for hiding this comment

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

Pull request overview

Copilot reviewed 1 out of 1 changed files in this pull request and generated 1 comment.

Comment thread .github/workflows/python.yml
@sebschmi sebschmi merged commit 841b8fc into main May 7, 2026
32 checks passed
@sebschmi sebschmi deleted the fix_python_release branch May 7, 2026 11:56
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.

2 participants