Skip to content

fix(cd): add SymbolPackageFormat=snupkg and safe glob for release assets#106

Closed
kieronlanning wants to merge 2 commits into
mainfrom
fix/cd-snupkg-glob
Closed

fix(cd): add SymbolPackageFormat=snupkg and safe glob for release assets#106
kieronlanning wants to merge 2 commits into
mainfrom
fix/cd-snupkg-glob

Conversation

@kieronlanning

Copy link
Copy Markdown
Contributor

Problem

The \Publish GitHub Release\ step failed with:

o matches found for ./artifacts/nuget/*.snupkg\

\dotnet pack --include-symbols\ without /p:SymbolPackageFormat=snupkg\ generates *.symbols.nupkg, not *.snupkg. The bash glob fails hard under \set -euo pipefail.

Fixes

  • Add /p:SymbolPackageFormat=snupkg\ to the pack command so .snupkg\ files are actually produced
  • Use \shopt -s nullglob\ to safely collect assets so the release create never fails on an empty glob

Copilot AI review requested due to automatic review settings May 24, 2026 16:37
@kieronlanning kieronlanning enabled auto-merge (squash) May 24, 2026 16:37

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

Adjusts the CD workflow packaging and release asset collection so GitHub releases reliably include symbol packages and don’t fail due to missing glob matches.

Changes:

  • Updates dotnet pack to emit .snupkg files by setting SymbolPackageFormat=snupkg.
  • Collects release assets via a nullglob-safe Bash array and passes them to gh release create.

Comment thread .github/workflows/cd.yml Outdated
- dotnet pack without SymbolPackageFormat=snupkg produces .symbols.nupkg, not .snupkg
- Use nullglob so the release create doesn't fail if a glob matches nothing

Co-authored-by: Copilot <223556219+Copilot@users.noreply.github.com>
auto-merge was automatically disabled May 24, 2026 17:19

Head branch was pushed to by a user without write access

@kieronlanning kieronlanning enabled auto-merge May 24, 2026 17:22
@kieronlanning kieronlanning added the skip-changeset PR does not need a changeset entry label May 24, 2026
@kieronlanning kieronlanning disabled auto-merge May 25, 2026 09:49
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

skip-changeset PR does not need a changeset entry

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants