-
Notifications
You must be signed in to change notification settings - Fork 10.4k
CopyCabs #10340
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
CopyCabs #10340
Conversation
This should copy CAB files if there are any @natemcmaster, I'm assuming that signing is done by the time we hit this target. |
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.
Straightforward 😺
@JunTaoLuo and @natemcmaster I believe this is just FYI for you two. @joeloff, yes, the cabs are signed before the copy you're changing. |
@joeloff the required checks are done. Let's get this in. Unless, that is, you're waiting for something specific in the Helix tests? |
- most likely cause of failing official builds - small typo in #10340
- most likely cause of failing official builds - small typo in #10340: need to find .cab files _below_ `$(TargetDir)`
- most likely cause of failing official builds - small typo in #10340: need to find .cab files _below_ `$(TargetDir)`
- most likely cause of failing official builds - small typo in #10340: need to find .cab files _below_ `$(TargetDir)`
Copy CAB files to artifacts