-
Notifications
You must be signed in to change notification settings - Fork 1.1k
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
Validate VMR Shipping Packages #45836
base: main
Are you sure you want to change the base?
Conversation
2ff6121
to
c818ebe
Compare
Test run (internal Microsoft link) |
Awesome!!!! |
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.
Approved with minor tweak if necessary.
Will this break official builds until the linked issue is resolved? Or is the error just ignored? |
The stage warns currently |
Closes dotnet/source-build#4068
When not all packages are signed, the validation stage is marked with a warning and an error message is outputted to the UI.
Note: Until dotnet/source-build#4678 is resolved, the validation stage will output the error that not all packages are signed. This is because mac & linux builds are currently doing dry-run signing.