Skip to content

Fix release flow#5579

Merged
jdetter merged 6 commits into
masterfrom
jdetter/fix-release-flow
Jul 24, 2026
Merged

Fix release flow#5579
jdetter merged 6 commits into
masterfrom
jdetter/fix-release-flow

Conversation

@jdetter

@jdetter jdetter commented Jul 22, 2026

Copy link
Copy Markdown
Contributor

Description of Changes

  • Fixes issues in the crates.io release and csharp release related to handling the version suffix.

API and ABI breaking changes

None

Expected complexity level and risk

1 - this is just a release change.

Testing

jdetter and others added 4 commits July 22, 2026 14:49
The release tag for a hotfix (e.g. v2.7.0-hotfix2) is passed straight
through to the C# release, but `dotnet pack` derives the NuGet package
version from the csproj <Version>, which is not bumped for a hotfix.
So pack produces SpacetimeDB.*.2.7.0.nupkg while the publish step looked
for ...2.7.0-hotfix2.nupkg and failed with "Package not found".

Strip the -hotfix<N> suffix when building the NuGet package paths (and
the summary output). We intentionally do not strip a generic prerelease
suffix like -rc1, since those are baked into the csproj <Version> and are
part of the real package version; only -hotfix is a tag-only concept.
The Unity tag push keeps the full hotfix version.

Co-Authored-By: Claude Opus 4.8 <noreply@anthropic.com>
Signed-off-by: John Detter <4099508+jdetter@users.noreply.github.com>

@rekhoff rekhoff 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.

Looks good to me.
Does the apt update and strips out everything after the - so we don't get conflicts in environments that don't like -hotfix suffixes.

@clockwork-labs-bot
clockwork-labs-bot added this pull request to the merge queue Jul 23, 2026
@github-merge-queue
github-merge-queue Bot removed this pull request from the merge queue due to failed status checks Jul 23, 2026
@jdetter
jdetter added this pull request to the merge queue Jul 24, 2026
Merged via the queue into master with commit 206de67 Jul 24, 2026
43 checks passed
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