Skip to content

[release/dev16.7-vs-deps] Update dependencies from dotnet/arcade#50432

Closed
dotnet-maestro[bot] wants to merge 9 commits intorelease/dev16.7-vs-depsfrom
darc-release/dev16.7-vs-deps-6a74678b-865f-4adc-aec5-5e4b92d55593
Closed

[release/dev16.7-vs-deps] Update dependencies from dotnet/arcade#50432
dotnet-maestro[bot] wants to merge 9 commits intorelease/dev16.7-vs-depsfrom
darc-release/dev16.7-vs-deps-6a74678b-865f-4adc-aec5-5e4b92d55593

Conversation

@dotnet-maestro
Copy link
Contributor

@dotnet-maestro dotnet-maestro bot commented Jan 13, 2021 •
edited
Loading

This pull request updates the following dependencies

From https://github.com/dotnet/arcade

  • Subscription: 0a85d329-8a2e-49f6-e608-08d81de7a64f
  • Build: 20210726.2
  • Date Produced: 7/26/2021 4:20 PM
  • Commit: c444d7967c5b79d16f2150370504368591883268
  • Branch: refs/heads/release/3.x

@dotnet-maestro
Update dependencies from https://github.com/dotnet/arcade build 20201…
4d75729 
…221.2

Microsoft.DotNet.Arcade.Sdk
 From Version 1.0.0-beta.20569.8 -> To Version 1.0.0-beta.20621.2
@dotnet-maestro dotnet-maestro bot requested a review from a team as a code owner January 13, 2021 16:51
ghost
ghost approved these changes Jan 13, 2021
View reviewed changes
Copy link

@ghost ghost left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Auto-approve

@dotnet-maestro
Update dependencies from https://github.com/dotnet/arcade build 20210…
6fb48da 
…113.4

Microsoft.DotNet.Arcade.Sdk
 From Version 1.0.0-beta.20569.8 -> To Version 1.0.0-beta.21063.4
@Cosifne
Copy link
Member

Cosifne commented Jan 22, 2021

Tag @JoeRobich,
Do you know how to get this in?

@JoeRobich
Copy link
Member

JoeRobich commented Jan 22, 2021

Do you know how to get this in?

@Cosifne Yeah the problem is this branch still references the deprecated myget feeds from the NuGet.config. I believe you should be able to update the NuGet.config to match master to get it building again.

@JoeRobich
Copy link
Member

JoeRobich commented Jan 23, 2021 •
edited
Loading

So I updated NuGet.config to match 'master' and also included the myget-legacy feed to have access to some older packages. I bumped the RoslynDiagnostics version to get the closest version of the Analyzer package that was available. I then cherry-picked a couple changes to RunTests from dev16.8 to remove the RestSharp dependency.

@JoeRobich JoeRobich force-pushed the darc-release/dev16.7-vs-deps-6a74678b-865f-4adc-aec5-5e4b92d55593 branch from ad33a06 to 03943e4 Compare January 23, 2021 01:31
@JoeRobich
Copy link
Member

JoeRobich commented Jan 23, 2021

Ok. Started over and took a different approach. Used the 16.8 nuget.config instead of master's. Added the myget-legacy feed to pull in some older packages. Bumped the RoslynDiagnostics version to get an available version of the Analyzers. Bumped the M.VisualStudio.Composition package to a version that wouldn't cause downgrades.

@runfoapp runfoapp bot mentioned this pull request Jan 25, 2021
Closed
@dotnet-maestro
Update dependencies from https://github.com/dotnet/arcade build 20210…
59a5da4 
…210.10

Microsoft.DotNet.Arcade.Sdk
 From Version 1.0.0-beta.20569.8 -> To Version 1.0.0-beta.21110.10
@dotnet-maestro
Update dependencies from https://github.com/dotnet/arcade build 20210…
bca3c20 
…622.2

Microsoft.DotNet.Arcade.Sdk
 From Version 1.0.0-beta.20569.8 -> To Version 1.0.0-beta.21322.2
@dotnet-maestro
Update dependencies from https://github.com/dotnet/arcade build 20210…
e925a27 
…709.1

Microsoft.DotNet.Arcade.Sdk
 From Version 1.0.0-beta.20569.8 -> To Version 1.0.0-beta.21359.1
@dotnet-maestro
Update dependencies from https://github.com/dotnet/arcade build 20210…
3752ea3 
…726.2

Microsoft.DotNet.Arcade.Sdk
 From Version 1.0.0-beta.20569.8 -> To Version 1.0.0-beta.21376.2
@JoeRobich JoeRobich closed this Aug 16, 2021
@JoeRobich JoeRobich deleted the darc-release/dev16.7-vs-deps-6a74678b-865f-4adc-aec5-5e4b92d55593 branch August 16, 2021 20:50
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

1 more reviewer
Reviewers whose approvals may not affect merge requirements

Assignees

No one assigned

Labels

Area-Infrastructure

Projects

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

3 participants

@Cosifne @JoeRobich @Dotnet-GitSync-Bot