Skip to content

Update NuGet publish workflow configuration#16

Merged
Serpensin merged 2 commits into
masterfrom
test
Sep 14, 2025
Merged

Update NuGet publish workflow configuration#16
Serpensin merged 2 commits into
masterfrom
test

Conversation

@Serpensin

Copy link
Copy Markdown
Owner

Added a release trigger for published types and enabled manual triggering of the workflow with a required version input for NuGet package version.

Added a release trigger for published types and enabled
manual triggering of the workflow with a required version
input for NuGet package version.
Copilot AI review requested due to automatic review settings September 14, 2025 21:35

Copilot AI 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.

Pull Request Overview

Updates the NuGet publish workflow to use GitHub release events instead of tag pushes, and adds manual workflow dispatch capability with version input. This provides more control over when packages are published and allows for manual publishing with custom versions.

  • Replaced tag-based trigger with release published event
  • Added workflow_dispatch trigger with required version input parameter
  • Maintained existing branch restriction logic
Comments suppressed due to low confidence (1)

.github/workflows/nuget-publish.yml:11

  • The default value for a required input should not be empty. Either make the input optional or provide a meaningful default version like '0.0.1'.
        default: ''

Tip: Customize your code reviews with copilot-instructions.md. Create the file or learn how to get started.

Changed the way the NuGet API key is accessed in the
`dotnet nuget push` command from a string to an
environment variable for improved security and correctness.
@sonarqubecloud

Copy link
Copy Markdown

@Serpensin Serpensin merged commit 172c212 into master Sep 14, 2025
6 checks passed
@Serpensin Serpensin deleted the test branch September 14, 2025 21:45
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