Skip to content

Commit

Permalink
Only push symbol packages for prereleases
Browse files Browse the repository at this point in the history
Symbol packages include the .dll alongside the .pdb, so if you don't
need the .pdb then the only downside is a larger-than-needed download.
But since prerelease packages are what you're most likely to be trying
to debug, it's helpful to have the .pdb included.

This also avoids the "Package 'Foo.0.2.1.symbols.nupkg' already
exists" error when the "Foo.0.2.1.nupkg" package was uploaded earlier in
the process.
  • Loading branch information
rmunn committed Jul 18, 2024
1 parent 945128c commit 3e12137
Showing 1 changed file with 1 addition and 1 deletion.
2 changes: 1 addition & 1 deletion .github/workflows/nuget-ci-cd.yml
Original file line number Diff line number Diff line change
Expand Up @@ -124,7 +124,7 @@ jobs:
if: github.event_name == 'pull_request' || (github.event_name == 'push' && startsWith(github.ref, 'refs/heads'))
shell: bash
run: |
dotnet nuget push "src/artifacts/package/release/*.nupkg" -s https://nuget.pkg.github.com/sillsdev/index.json -k "${{ secrets.GITHUB_TOKEN }}" --skip-duplicate
dotnet nuget push "src/artifacts/package/release/*.symbols.nupkg" -s https://nuget.pkg.github.com/sillsdev/index.json -k "${{ secrets.GITHUB_TOKEN }}" --skip-duplicate
- name: Publish package to NuGet.org
if: github.event_name == 'push' && startsWith(github.ref, 'refs/tags')
Expand Down

0 comments on commit 3e12137

Please sign in to comment.