-
Notifications
You must be signed in to change notification settings - Fork 545
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
[master] Update dependencies from dotnet/arcade #1678
[master] Update dependencies from dotnet/arcade #1678
Conversation
…528.4 Microsoft.DotNet.Arcade.Sdk From Version 5.0.0-beta.20261.9 -> To Version 5.0.0-beta.20278.4
Auto-Merge StatusThis pull request has not been merged because Maestro++ is waiting on the following merge policies.
|
@ajcvickers Can you find someone to look into this?
|
Auto-Merge StatusThis pull request has not been merged because Maestro++ is waiting on the following merge policies.
|
@smitpatel @bricelam Any ideas? |
Auto-Merge StatusThis pull request has not been merged because Maestro++ is waiting on the following merge policies.
|
I investigated a bit into this.
Perhaps someone from runtime/SDK team should check and figure out if the SDK has made breaking change and guide how to change our code according to it. |
Auto-Merge StatusThis pull request has not been merged because Maestro++ is waiting on the following merge policies.
|
I checked out this PR locally and looked at artifacts\obj\ef6\Release\net40\ef6.AssemblyInfo.cs. It's not the same things as SharedAssemblyInfo.cs. It's a file "Generated by the MSBuild WriteCodeFragment class." So I then added @smitpatel I opened #1680 to include SharedAssemblyInfo.cs in ef6.csrproj. Does this make sense? |
Auto-Merge StatusThis pull request has not been merged because Maestro++ is waiting on the following merge policies.
|
@smitpatel It looks like we have a new issue. Can you or someone else take a look at what's causing the test failure?
|
Auto-Merge StatusThis pull request has not been merged because Maestro++ is waiting on the following merge policies.
|
It looks like this is just a flaky test, so I'll rerun. @ajcvickers Is there a test quarantine process for this repo? |
@halter73 - Thanks Stephen for investigating earlier issue. I will look into the test failure. |
Auto-Merge StatusThis pull request will be merged because the following merge policies have succeeded.
|
3e4264f
to
33163a5
Compare
This pull request updates the following dependencies
From https://github.com/dotnet/arcade