Skip to content

Configure nuget-release workflow to validate PRs without publishing#1099

Merged
adamhathcock merged 2 commits intoadam/remove-old-releasefrom
copilot/sub-pr-1098
Jan 3, 2026
Merged

Configure nuget-release workflow to validate PRs without publishing#1099
adamhathcock merged 2 commits intoadam/remove-old-releasefrom
copilot/sub-pr-1098

Conversation

Copy link
Contributor

Copilot AI commented Jan 3, 2026

The nuget-release.yml workflow only ran on direct pushes to master/release or tags, meaning PRs weren't validated by CI before merge.

Workflow Changes:

  • Added pull_request trigger for master and release branches
  • Gated NuGet publishing step with github.event_name != 'pull_request' condition

Behavior:

  • PRs: Build, test, and upload artifacts for validation - no publishing
  • Merges/Tags: Full workflow including NuGet publishing
on:
  push:
    branches: ['master', 'release']
    tags: ['[0-9]+.[0-9]+.[0-9]+']
  pull_request:
    branches: ['master', 'release']

# ...

- name: Push to NuGet
  if: success() && matrix.os == 'windows-latest' && github.event_name != 'pull_request'

💬 We'd love your input! Share your thoughts on Copilot coding agent in our 2 minute survey.

Co-authored-by: adamhathcock <527620+adamhathcock@users.noreply.github.com>
Copilot AI changed the title [WIP] Remove old GitHub Actions workflow and update solution file Configure nuget-release workflow to validate PRs without publishing Jan 3, 2026
Copilot AI requested a review from adamhathcock January 3, 2026 14:16
@adamhathcock adamhathcock marked this pull request as ready for review January 3, 2026 14:22
@adamhathcock adamhathcock merged commit 574d9f9 into adam/remove-old-release Jan 3, 2026
@adamhathcock adamhathcock deleted the copilot/sub-pr-1098 branch January 3, 2026 14:22
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

Comments