Skip to content

Fix non-final releases creating final-release git tags #9085

New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Merged
merged 2 commits into from
Feb 19, 2025

Conversation

Wumpf
Copy link
Member

@Wumpf Wumpf commented Feb 19, 2025

Related

What

0.22.0 GH release & branch was incorrectly created on the RC1 commit

This causes all our release artifacts (and thus cargo binstall) to point at RC1.
This is a bit of a problem as it messes with all sort of things. Luckily the main release artifact that people download from there is the C++ SDK which is close not not affected at all (it does have the RC1 version backed in there though as well, so it doesn't line up with e.g. python installed viewer)

This was triggered by a mistake in a PR right after 0.21 which caused us to create non-rc git tags on rc builds which then caused failures on the actual release. Those were missed, very likely because we had other spurious issues

@Wumpf Wumpf added 🪳 bug Something isn't working ⛴ release Related to shipping or publishing exclude from changelog PRs with this won't show up in CHANGELOG.md labels Feb 19, 2025
@Wumpf Wumpf changed the title Fix non-final builds creating final git tags Fix non-final releases creating final-release git tags Feb 19, 2025
Co-authored-by: Jan Procházka <[email protected]>
@Wumpf Wumpf merged commit ddf3897 into main Feb 19, 2025
31 checks passed
@Wumpf Wumpf deleted the andreas/fix-release-tagging branch February 19, 2025 17:07
oxkitsune pushed a commit that referenced this pull request Feb 19, 2025
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
🪳 bug Something isn't working exclude from changelog PRs with this won't show up in CHANGELOG.md ⛴ release Related to shipping or publishing
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants