-
Notifications
You must be signed in to change notification settings - Fork 816
[release/3.0] Update dependencies from dotnet/arcade #2397
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 #2397
Conversation
…924.3 - Microsoft.DotNet.Arcade.Sdk - 1.0.0-beta.19474.3 - Microsoft.DotNet.GenAPI - 1.0.0-beta.19474.3 - Microsoft.DotNet.Helix.Sdk - 2.0.0-beta.19474.3
Auto-Merge StatusThis pull request has not been merged because Maestro++ is waiting on the following merge policies.
|
Same as dotnet/blazor#1899 |
Auto-Merge StatusThis pull request will be merged because the following merge policies have succeeded.
|
<!--Begin: Package sources managed by Dependency Flow automation. Do not edit the sources below.--> | ||
<add key="darc-pub-dotnet-core-setup-7d57652" value="https://pkgs.dev.azure.com/dnceng/public/_packaging/darc-pub-dotnet-core-setup-7d57652f/nuget/v3/index.json" /> | ||
<add key="darc-pub-dotnet-corefx-4ac4c03" value="https://pkgs.dev.azure.com/dnceng/public/_packaging/darc-pub-dotnet-corefx-4ac4c036/nuget/v3/index.json" /> | ||
<!--End: Package sources managed by Dependency Flow automation. Do not edit the sources above.--> | ||
<clear /> |
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.
@mmitche I'm a bit surprised this PR validated properly given I've seen mention of moving the <clear />
arround being a bug. Did this update succeed only because Extensions has already moved on from GA packages?
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.
Because all the packages (stable) that were in those feeds have been published to nuget.org.
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.
Is the dependency update code smart enough to move the <clear />
back up when it next changes the feeds in the comments?
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.
Yes
This pull request updates the following dependencies
From https://github.com/dotnet/arcade