Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Switch to snupkg package format and publish symbols to nuget.org #143

Merged
merged 3 commits into from
Aug 9, 2021

Conversation

yousif-bugsnag
Copy link
Contributor

Goal

Currently Appveyor attempts to upload the nupkg build artifacts to nuget.org and the .symbols.nupkg artifacts to the 3rd party symbol server nuget.symbsrc.net, which is no longer maintained. NuGet now supports publishing symbol packages directly to NuGet, using the new snupkg package format.

Design

Updates the build to create NuGet symbol packages using the new .snupkg format rather than the now legacy .symbols.nupkg format, and configures Appveyor publish these to nuget.org rather than the 3rd-party smbsrc.net.

appveyor.yml Outdated Show resolved Hide resolved
twometresteve
twometresteve previously approved these changes Aug 2, 2021
Copy link
Contributor

@twometresteve twometresteve left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM, from what I can tell at least!

@yousif-bugsnag
Copy link
Contributor Author

After looking at the appveyor build log from the previous commit, I noticed that while the snupkg artifacts were being created they were not being found and uploaded by appveyor.

I've tweaked the wildcard path for the build artifacts so that these are now picked up, and confirmed these are now being uploaded by checking the appveyor build log for this commit.

@yousif-bugsnag yousif-bugsnag merged commit 353a055 into next Aug 9, 2021
@yousif-bugsnag yousif-bugsnag deleted the yousif/update-nuget-symbol-package branch August 9, 2021 08:26
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.

None yet

2 participants