-
-
Notifications
You must be signed in to change notification settings - Fork 35
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
Port to Ionide.KeepAChangelog #132
Conversation
Nice. Thoughts on putting the packaging props into a |
@@ -126,14 +98,12 @@ Target.create | |||
"Pack" | |||
(fun _ -> | |||
let properties = | |||
[ ("Version", nugetVersion) | |||
("Authors", authors) | |||
[ ("Authors", authors) | |||
("PackageProjectUrl", gitUrl) | |||
("PackageTags", tags) | |||
("RepositoryType", "git") |
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.
these source control properties can be removed - sourcelink/dotnet.reproducible builds infer them correctly for you.
("PackageProjectUrl", gitUrl) | ||
("PackageTags", tags) | ||
("RepositoryType", "git") | ||
("RepositoryUrl", gitUrl) | ||
("PackageLicenseExpression", "MIT") | ||
("PackageReleaseNotes", packageReleaseNotes) | ||
("PackageDescription", summary) | ||
("EnableSourceLink", "true") ] |
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.
this one can be removed as well - sourcelink is just auto-on with the dotnet.reproduciblebuild package.
@@ -126,14 +98,12 @@ Target.create | |||
"Pack" | |||
(fun _ -> | |||
let properties = | |||
[ ("Version", nugetVersion) | |||
("Authors", authors) | |||
[ ("Authors", authors) | |||
("PackageProjectUrl", gitUrl) |
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.
you can remove this project url as well - it defaults to the repo git url that sourcelink derives from the origin
uri
This reverts commit 921703b.
This reverts commit 921703b.
Use Ionide.KeepAChangelog instead of custom FAKE code.