-
Notifications
You must be signed in to change notification settings - Fork 263
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
Support Sourcelink and Deterministic Build #737
Support Sourcelink and Deterministic Build #737
Conversation
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Thanks very much for this! 🙇
Have left some newbie questions for you before I merge.
This one is still on draft as it isn't tested yet. I have applied many of these changes in the past, but every build pipeline is somewhat different. |
RepositoryUrl + RepositoryType
@dtchepak how is nuget package publish process organized?
|
f5d8852
to
c99f583
Compare
971b14a
to
b0ac750
Compare
@dtchepak this is ready, I also updated the description of this PR. Please note:
|
ba78b82
to
b1f0dd6
Compare
@304NotModified Thank you for the PR! |
Good question, I don't this this is needed:
https://devblogs.microsoft.com/dotnet/producing-packages-with-source-link/ |
polite bump @dtchepak :) |
It's done manually, normally I use AppVeyor build. (Release procedure). I'd love to have this automated at some point. |
Thanks @304NotModified ! |
@dtchepak could we please release this? :) |
I was going to just have this bundled in next feature release. Is there something this provides that is blocking you? |
5.2 or 5.1.1 are both fine to me. I prefer 5.2 as it isnt a bugfix. |
Hi @dtchepak, as Source Link could help with better bug reports and deterministic build in security, I still hope to see a release of this. I like to contribute more but the delay of the releases is holding me back. |
@304NotModified MR for release 5.2 here: #791 I'll release it once I get core team approval 👍 |
This PR enables Source Link and Deterministic Builds.
Why
Most of the top NuGet packages has enabled Source Link. Also some are using Deterministic Build.
E.g. Newtonsoft.Json
I've also configured deterministic builds for other open-source projects, e.g. nunit/nunit#3810, App-vNext/Polly#839
More info
Before
(Checked with https://github.com/NuGetPackageExplorer/NuGetPackageExplorer)
After
Note, no signature because this is from local. You could also see this by downloading the .nugpkg and .snupkg from the artifacts from AppVeyor
Note! the .snukpg should be published to nuget.org, otherwise the source link won't work