build fixes

src/Polly.Extensions/Utils/OptionsReloadHelper.cs

@@ -2,12 +2,14 @@
 
 namespace Polly.Extensions.Utils;
 
-internal sealed class OptionsReloadHelper<T> : IDisposable
+#pragma warning disable CA1001 // we can get away of not disposing this class because it's active for a lifetime of app
+#pragma warning disable S2931
+
+internal sealed class OptionsReloadHelper<T>
 {
-    private readonly IDisposable? _listener;
     private CancellationTokenSource _cancellation = new();
 
-    public OptionsReloadHelper(IOptionsMonitor<T> monitor, string name) => _listener = monitor.OnChange((_, changedNamed) =>
+    public OptionsReloadHelper(IOptionsMonitor<T> monitor, string name) => monitor.OnChange((_, changedNamed) =>
     {
         if (name == changedNamed)
         {
@@ -17,12 +19,6 @@ public OptionsReloadHelper(IOptionsMonitor<T> monitor, string name) => _listener
 
     public CancellationToken GetCancellationToken() => _cancellation.Token;
 
-    public void Dispose()
-    {
-        _cancellation.Dispose();
-        _listener?.Dispose();
-    }
-
     private void HandleChange()
     {
         var oldCancellation = _cancellation;

test/Polly.Extensions.Tests/Utils/OptionsReloadHelperTests.cs

@@ -15,7 +15,7 @@ public void Ctor_NamedOptions()
             .Setup(m => m.OnChange(It.IsAny<Action<string, string?>>()))
             .Returns(() => Mock.Of<IDisposable>());
 
-        using var helper = new OptionsReloadHelper<string>(monitor.Object, "name");
+        var helper = new OptionsReloadHelper<string>(monitor.Object, "name");
 
         monitor.VerifyAll();
     }