Supress S4039 and CA1010 warnings (#2329)

src/Polly/Context.Dictionary.cs

@@ -1,6 +1,6 @@
 namespace Polly;
 
-#pragma warning disable CA1033 // Interface methods should be callable by child types
+#pragma warning disable CA1033,S4039 // Interface methods should be callable by child types
 
 /// <summary>
 /// Context that carries with a single execution through a Policy.   Commonly-used properties are directly on the class.  Backed by a dictionary of string key / object value pairs, to which user-defined values may be added.

src/Polly/Context.cs

@@ -4,6 +4,7 @@ namespace Polly;
 /// Context that carries with a single execution through a Policy.   Commonly-used properties are directly on the class.  Backed by a dictionary of string key / object value pairs, to which user-defined values may be added.
 /// <remarks>Do not re-use an instance of <see cref="Context"/> across more than one call through .Execute(...) or .ExecuteAsync(...).</remarks>
 /// </summary>
+#pragma warning disable CA1010 // Collections should implement generic interface
 #pragma warning disable CA1710 // Identifiers should have correct suffix
 public partial class Context
 #pragma warning restore CA1710

src/Polly/Polly.csproj

@@ -7,8 +7,8 @@
     <ProjectType>Library</ProjectType>
     <MutationScore>70</MutationScore>
     <IncludePollyUsings>true</IncludePollyUsings>
-    <NoWarn>$(NoWarn);CA1010;CA1031;CA1051;CA1063;CA1064;CA1724;</NoWarn>
-    <NoWarn>$(NoWarn);S2223;S3215;S4039</NoWarn>
+    <NoWarn>$(NoWarn);CA1031;CA1051;CA1063;CA1064;CA1724;</NoWarn>
+    <NoWarn>$(NoWarn);S2223;S3215;</NoWarn>
     <!--Public API Analyzers: We do not need to fix these as it would break compatibility with released Polly versions-->
     <NoWarn>$(NoWarn);RS0037;</NoWarn>
   </PropertyGroup>