Skip to content

ci: Improve consistency for nuget publish - #177

Merged
guitarrapc merged 1 commit into
mainfrom
ci
Oct 6, 2026
Merged

guitarrapc merged 1 commit into
mainfrom
ci

Conversation

@guitarrapc

Copy link
Copy Markdown
Contributor

No description provided.

Copilot AI balanced review requested due to automatic review settings October 6, 2026 08:24
@guitarrapc
guitarrapc requested a review from mayuki as a code owner October 6, 2026 08:24

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

The concurrency configuration can cancel pending releases when additional tags are dispatched.

Review effort: Balanced
Findings: 1 High severity

Open (1)
What changed in this PR

Improves release coordination through the shared Cysharp workflows.

Changes:

  • Adds release validation and passes the validated tag downstream.
  • Serializes release runs and simplifies release inputs and permissions.
  • Delays branch cleanup until release creation finishes, including failure paths.
File Description
.github/​workflows/​build-release.yaml Updates release validation, concurrency, dependencies, and cleanup.

💡 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
@guitarrapc
guitarrapc merged commit be501de into main Oct 6, 2026
22 checks passed
@guitarrapc
guitarrapc deleted the ci branch October 6, 2026 08:37
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