Skip to content

ci: new release flow by handle publish package after create-rele - #137

Merged
guitarrapc merged 2 commits into
mainfrom
ci
Sep 30, 2026
Merged

guitarrapc merged 2 commits into
mainfrom
ci

Conversation

@guitarrapc

Copy link
Copy Markdown
Contributor

Summary

Previously, NuGet packages could be published before release tag validation and GitHub release creation had completed.

This change ensures that dotnet nuget push runs only after the tag has been validated and create-release has succeeded.

Copilot AI left a comment

Copy link
Copy Markdown

Choose a reason for hiding this comment

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

Copilot review overview

🟡 Changes recommended

Symbol publishing and retry behavior can leave releases incomplete.

Review effort: Balanced
Findings: 2 Medium severity

Open (2)
What changed in this PR

Restructures the release workflow so package publication follows validation, builds, and release creation.

Changes:

  • Adds release validation and serialized execution.
  • Builds packages from the version-update commit.
  • Defers Rust and NuGet publication until after release creation.
File Description
.github/​workflows/​build-release.yaml Reorganizes validation, building, release, publishing, and cleanup jobs.

💡 Add a code-review agent skill or configure MCP servers for context-aware, tailored reviews. Learn more in the docs.

Comment thread .github/workflows/build-release.yaml
Comment thread .github/workflows/build-release.yaml Outdated
@guitarrapc
guitarrapc merged commit d924624 into main Sep 30, 2026
7 checks passed
@guitarrapc
guitarrapc deleted the ci branch September 30, 2026 09:21
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