Skip to content

ci(bindings/dotnet): drive releases via workflow_dispatch with previe…#7422

Merged
tisonkun merged 2 commits into
apache:mainfrom
Fatorin:dotnet-manual-publish
Apr 27, 2026
Merged

ci(bindings/dotnet): drive releases via workflow_dispatch with previe…#7422
tisonkun merged 2 commits into
apache:mainfrom
Fatorin:dotnet-manual-publish

Conversation

@Fatorin
Copy link
Copy Markdown
Contributor

@Fatorin Fatorin commented Apr 21, 2026

Rationale for this change

The initial release_dotnet.yml used a shared v* tag trigger, which would fire on core and other bindings' tags as well.
Since NuGet packages cannot be deleted (only unlisted) and NuGet has no staging repository, any accidental publish is irreversible.

After discussion with a maintainer, releases are driven entirely via workflow_dispatch — every publish is an explicit, intentional action.

What changes are included in this PR?

  • Remove the push: tags: trigger.
  • Replace the dotnet_publish boolean with a release_type choice input (none / preview / rc).
  • Switch .csproj from <Version> to <VersionPrefix> and use dotnet pack --version-suffix to assemble the final version.
release_type Version Publishes to
none (default) {VersionPrefix}-preview.{run_number} GitHub Actions artifact only (dry run)
preview {VersionPrefix}-preview.{run_number} nuget.org
rc {VersionPrefix}-rc nuget.org

Are there any user-facing changes?

No.

AI Usage Statement

This PR was developed with assistance from Claude (Anthropic) for research and drafting.

@Fatorin Fatorin requested review from Xuanwo and tisonkun as code owners April 21, 2026 16:06
@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 21, 2026
Comment thread bindings/dotnet/OpenDAL/OpenDAL.csproj Outdated
@dosubot dosubot Bot added the lgtm This PR has been approved by a maintainer label Apr 27, 2026
@tisonkun tisonkun merged commit d69cde5 into apache:main Apr 27, 2026
50 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

lgtm This PR has been approved by a maintainer 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