Skip to content

Cicd wrong prerelease tag#778

Merged
aeppling merged 2 commits intodevelopfrom
cicd-wrong-prerelease-tag
Mar 21, 2026
Merged

Cicd wrong prerelease tag#778
aeppling merged 2 commits intodevelopfrom
cicd-wrong-prerelease-tag

Conversation

@aeppling
Copy link
Contributor

Summary

Test plan

  • cargo fmt --all && cargo clippy --all-targets && cargo test
  • Manual testing: rtk <command> output inspected

Important: All PRs must target the develop branch (not master).
See CONTRIBUTING.md for details.

Was using develop ancestor -> wrong behavior because version happen on master branch
@pszymkowiak pszymkowiak added bug Something isn't working effort-small Quelques heures, 1 fichier labels Mar 21, 2026
@pszymkowiak
Copy link
Collaborator

[w] wshm · Automated triage by AI

📊 Automated PR Analysis

🐛 Type bug-fix
🟡 Risk medium

Summary

Fixes the CI/CD version computation to use the globally latest stable tag instead of only considering tags that are ancestors of HEAD. This resolves an issue where the prerelease tag was computed incorrectly because the previous logic filtered tags by ancestry, potentially missing the actual latest release.

Review Checklist

  • Tests present
  • Breaking change
  • Docs updated

Analyzed automatically by wshm · This is an automated analysis, not a human review.

@aeppling aeppling merged commit 4b52bfe into develop Mar 21, 2026
10 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

bug Something isn't working effort-small Quelques heures, 1 fichier

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants