Bump Roslynator.Analyzers from 4.7.0 to 4.8.0 (#839)

* Bump Roslynator.Analyzers from 4.7.0 to 4.8.0

Bumps [Roslynator.Analyzers](https://github.com/dotnet/roslynator) from 4.7.0 to 4.8.0.
- [Release notes](https://github.com/dotnet/roslynator/releases)
- [Changelog](https://github.com/dotnet/roslynator/blob/main/ChangeLog.md)
- [Commits](dotnet/roslynator@v4.7.0...v4.8.0)

---
updated-dependencies:
- dependency-name: Roslynator.Analyzers
  dependency-type: direct:production
  update-type: version-update:semver-minor
...

Signed-off-by: dependabot[bot] <support@github.com>

* dotnet format from workflow for PR 839

* dotnet format from workflow for PR 839

* Revert "dotnet format from workflow for PR 839"

This reverts commit 396493c.

* Revert "dotnet format from workflow for PR 839"

This reverts commit b7e0805.

* Temporarily disable RCS1251

---------

Signed-off-by: dependabot[bot] <support@github.com>
Co-authored-by: dependabot[bot] <49699333+dependabot[bot]@users.noreply.github.com>
Co-authored-by: dependabot[bot] <dependabot[bot]@users.noreply.github.com>
Co-authored-by: edumserrano <edumserrano@users.noreply.github.com>
Co-authored-by: Eduardo Serrano <eduardomserrano@outlook.com>

Directory.Build.props

@@ -51,7 +51,7 @@
       <PrivateAssets>all</PrivateAssets>
       <IncludeAssets>runtime; build; native; contentfiles; analyzers; buildtransitive</IncludeAssets>
     </PackageReference>
-    <PackageReference Include="Roslynator.Analyzers" Version="4.7.0">
+    <PackageReference Include="Roslynator.Analyzers" Version="4.8.0">
       <PrivateAssets>all</PrivateAssets>
       <IncludeAssets>runtime; build; native; contentfiles; analyzers; buildtransitive</IncludeAssets>
     </PackageReference>

src/DotNet.Sdk.Extensions/Polly/Http/Resilience/Events/IResiliencePoliciesEventHandler.cs

@@ -3,10 +3,15 @@ namespace DotNet.Sdk.Extensions.Polly.Http.Resilience.Events;
 /// <summary>
 /// Defines the events produced by the HttpClient's resilience policies.
 /// </summary>
+#pragma warning disable RCS1251 // Remove unnecessary braces from record declaration
+// Shouldn't have to disable RCS1251 but if I fix it then the StyleCopAnalyzers starts
+// crashing. For now I'm ignoring RCS1251 hoping the StyleCopAnalyzers get updated
+// soon, if not, consider removing the StyleCopAnalyzers
 public interface IResiliencePoliciesEventHandler :
     ITimeoutPolicyEventHandler,
     IRetryPolicyEventHandler,
     ICircuitBreakerPolicyEventHandler,
     IFallbackPolicyEventHandler
 {
 }
+#pragma warning restore RCS1251 // Remove unnecessary braces from record declaration

src/DotNet.Sdk.Extensions/Polly/Http/Resilience/Extensions/ResiliencePoliciesHttpClientBuilderExtensions.cs

@@ -5,9 +5,14 @@ namespace DotNet.Sdk.Extensions.Polly.Http.Resilience.Extensions;
 /// </summary>
 public static class ResiliencePoliciesHttpClientBuilderExtensions
 {
+    // Shouldn't have to disable RCS1251 but if I fix it then the StyleCopAnalyzers starts
+    // crashing. For now I'm ignoring RCS1251 hoping the StyleCopAnalyzers get updated
+    // soon, if not, consider removing the StyleCopAnalyzers
+#pragma warning disable RCS1251 // Remove unnecessary braces from record declaration
     private sealed class BlankHttpMessageHandler : DelegatingHandler
     {
     }
+#pragma warning restore RCS1251 // Remove unnecessary braces from record declaration
 
     /// <summary>
     /// Adds resilience policies to the <see cref="HttpClient"/>.