Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
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
Add SdkArchiveDiff task to verify the sdk archive has all the expected files #18748
Add SdkArchiveDiff task to verify the sdk archive has all the expected files #18748
Changes from 15 commits
e894991
cd1b907
a4bee91
5cc5ee0
2862e1f
a336dd0
a3256e2
8c32f67
7b16584
0eea7e0
8a5244c
d30c740
f704a53
45728f7
b6fba7e
9ffe5c2
7632ab4
3cf9a38
478bb28
f843239
3ed9096
9b43ca3
2b3389d
eafa1ba
5a2d878
8eb0cd2
3adb0cc
97d8e4d
d49fa67
4f6bac3
2075e87
3c848d5
97fc3b8
36d3cfa
2c5bee4
b290188
60c0a7e
6ca4c6f
2286d77
File filter
Filter by extension
Conversations
Jump to
There are no files selected for viewing
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.
Given the complexity of the version numbers that show up in the set of assets produced by .NET, we created a version identifier a long time ago. It's what powers creation of the aka.ms links: https://github.com/dotnet/arcade/blob/main/src/Microsoft.DotNet.VersionTools/lib/src/BuildManifest/VersionIdentifier.cs#L51
I'm not totally sure how well it works for paths inside an SDK, but it may be worth looking at if you start to run into corner cases with the checker.
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 for the link! I copied that file to this project. I tried referencing that package, but it references a different version of MSBuild that Installer uses and that caused build issues.