Skip to content

ci: skip dotnet publish for rc tags#7433

Closed
Xuanwo wants to merge 1 commit into
mainfrom
xuanwo/fix-dotnet-release-rc-publish
Closed

ci: skip dotnet publish for rc tags#7433
Xuanwo wants to merge 1 commit into
mainfrom
xuanwo/fix-dotnet-release-rc-publish

Conversation

@Xuanwo
Copy link
Copy Markdown
Member

@Xuanwo Xuanwo commented Apr 27, 2026

Which issue does this PR close?

N/A.

Rationale for this change

The v0.56.0-rc.4 tag build showed that the .NET release workflow still tries to publish to NuGet for RC tags. RC tag builds should validate and package artifacts, but publishing to NuGet should only happen for final release tags or explicit manual publish runs.

What changes are included in this PR?

This PR gates the .NET publish job so prerelease tags such as v0.56.0-rc.4 skip NuGet publishing while final v* tags and workflow_dispatch publish runs keep the existing publish path.

Are there any user-facing changes?

No.

AI Usage Statement

Assisted.

@Xuanwo Xuanwo marked this pull request as ready for review April 27, 2026 11:13
@Xuanwo Xuanwo requested a review from tisonkun as a code owner April 27, 2026 11:13
@dosubot dosubot Bot added size:XS This PR changes 0-9 lines, ignoring generated files. releases-note/ci The PR modifies CI-related content or has a title that begins with "ci" labels Apr 27, 2026
Copy link
Copy Markdown
Member

@tisonkun tisonkun left a comment

Choose a reason for hiding this comment

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

#7422 should work for this.

@tisonkun
Copy link
Copy Markdown
Member

Closed as #7422 merged.

@tisonkun tisonkun closed this Apr 27, 2026
@Xuanwo Xuanwo deleted the xuanwo/fix-dotnet-release-rc-publish branch May 12, 2026 04:16
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

releases-note/ci The PR modifies CI-related content or has a title that begins with "ci" size:XS This PR changes 0-9 lines, ignoring generated files.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants