[main] Update dependencies from dotnet/arcade

[dotnet-maestro]

This pull request updates the following dependencies

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

• Subscription: a0cb50dc-7d38-4a32-e166-08db18b91f66
• Build: 20230302.1
• Date Produced: March 2, 2023 6:55:33 PM UTC
• Commit: 6b3afa7543dd5d69e621015cd1569fb0960b5f24
• Branch: refs/heads/main

• Updates:
  • Microsoft.DotNet.Arcade.Sdk: from 8.0.0-beta.22524.5 to 8.0.0-beta.23152.1

[ghost]

Auto-approving dependency update.

[ryanbrandenburg]

Looks like it wants us to bump the analyzer versions.

[jaredpar]

The way this target works, we're going to get a new warning every time the SDK updates

  <!-- Target to report a warning when SDK NetAnalyzers version is higher than the referenced NuGet NetAnalyzers version -->
  <Target Name="_ReportUpgradeNetAnalyzersNuGetWarning" BeforeTargets="CoreCompile" Condition="'$(_SkipUpgradeNetAnalyzersNuGetWarning)' != 'true' ">
    <Warning Text ="The .NET SDK has newer analyzers with version '$(_NETAnalyzersSDKAssemblyVersion)' than what version '$(_NETAnalyzersNuGetAssemblyVersion)' of 'Microsoft.CodeAnalysis.NetAnalyzers' package provides. Update or remove this package reference."
             Condition="'$(_NETAnalyzersNuGetAssemblyVersion)' != '' AND
                         '$(_NETAnalyzersSDKAssemblyVersion)' != '' AND
                          $(_NETAnalyzersNuGetAssemblyVersion) &lt; $(_NETAnalyzersSDKAssemblyVersion)"/>
  </Target>

Feel like we should just stop manually referencing this and take whatever the SDK is giving us. Is there a way to stop pinning the analyzers here and just use the default? Maybe just delete the package reference entirely.

[jaredpar]

@dotnet/razor-compiler, @dotnet/razor-tooling PTAL at the following commit which is all the changes needed to get this arcade merge going green. Will add some context to the ones I feel are more questionable

35a2366

[jjonescz]

I guess this is unnecessary since it's already set in the root editorconfig? But anyways, don't we want to have it enabled? At least eventually.

[jaredpar]

It's necessary though because this .editorconfig lists itself as a root so we don't actually consider the one above for files below.

[jaredpar]

Sigh ... looks like the tests are hanging on Windows now. Debugging.