Fix CA2211 (#2316)

Contributes to #1290.
App-vNext · Sep 28, 2024 · 856bfdf · 856bfdf
1 parent 9f004a2
commit 856bfdf
Show file tree

Hide file tree

Showing 3 changed files with 5 additions and 1 deletion.
diff --git a/src/Polly/Polly.csproj b/src/Polly/Polly.csproj
@@ -7,7 +7,7 @@
     <ProjectType>Library</ProjectType>
     <MutationScore>70</MutationScore>
     <IncludePollyUsings>true</IncludePollyUsings>
-    <NoWarn>$(NoWarn);CA1010;CA1031;CA1051;CA1063;CA1064;CA1724;CA1816;CA2211</NoWarn>
+    <NoWarn>$(NoWarn);CA1010;CA1031;CA1051;CA1063;CA1064;CA1724;CA1816;</NoWarn>
     <NoWarn>$(NoWarn);S2223;S3215;S4039</NoWarn>
     <!--Public API Analyzers: We do not need to fix these as it would break compatibility with released Polly versions-->
     <NoWarn>$(NoWarn);RS0037;</NoWarn>

diff --git a/src/Polly/Utilities/SystemClock.cs b/src/Polly/Utilities/SystemClock.cs
@@ -1,6 +1,8 @@
 #nullable enable
 namespace Polly.Utilities;
 
+#pragma warning disable CA2211 // Non-constant fields should not be visible
+
 /// <summary>
 /// Time related delegates used to support different compilation targets and to improve testability of the code.
 /// </summary>

diff --git a/src/Polly/Utilities/TaskHelper.cs b/src/Polly/Utilities/TaskHelper.cs
@@ -6,8 +6,10 @@ namespace Polly.Utilities;
 /// </summary>
 public static class TaskHelper
 {
+#pragma warning disable CA2211 // Non-constant fields should not be visible
     /// <summary>
     /// Defines a completed Task for use as a completed, empty asynchronous delegate.
     /// </summary>
     public static Task EmptyTask = Task.CompletedTask;
+#pragma warning restore
 }