-
Notifications
You must be signed in to change notification settings - Fork 10.4k
[release/3.0] Update dependencies from dotnet/arcade aspnet/Blazor #17270
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
[release/3.0] Update dependencies from dotnet/arcade aspnet/Blazor #17270
Conversation
…119.2 - Microsoft.DotNet.Arcade.Sdk - 1.0.0-beta.19569.2 - Microsoft.DotNet.GenAPI - 1.0.0-beta.19569.2 - Microsoft.DotNet.Helix.Sdk - 2.0.0-beta.19569.2
Auto-Merge StatusThis pull request has not been merged because Maestro++ is waiting on the following merge policies.
|
…120.2 - Microsoft.AspNetCore.Blazor.Mono - 3.0.0-preview9.19570.2
Test failure,
@pranavkm @SteveSandersonMS, have you seen this before? Can you take a look? |
Auto-Merge StatusThis pull request has not been merged because Maestro++ is waiting on the following merge policies.
|
Yes, this test has been a problem for us for a long time. The most recent person looking into it (there have been many) is @ryanbrandenburg (as in https://github.com/aspnet/AspNetCore-Internal/issues/3295#issuecomment-555196029). Ryan, are you planning to mark this test as flaky? |
Auto-Merge StatusThis pull request has not been merged because Maestro++ is waiting on the following merge policies.
|
I don't have much context on tests like |
Auto-Merge StatusThis pull request has not been merged because Maestro++ is waiting on the following merge policies.
|
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.
Not merging 'til @mmitche confirms the file deletions are correct and the test failures are resolved
Should mention I retried the failed jobs in https://dev.azure.com/dnceng/public/_build/results?buildId=433505 Hopefully the next attempt won't hit a different flaky-but-not-so-marked test failure. |
Auto-Merge StatusThis pull request will be merged because the following merge policies have succeeded.
|
Got this in because @mmitche confirmed elsewhere that deleting a few eng/common/** files all good and the validation build completed successfully. |
This pull request updates the following dependencies
From https://github.com/dotnet/arcade
From https://github.com/aspnet/Blazor