Resolving comments

Author: rossgrambo

examples/VariantAndAzureMonitorDemo/LoggerExtensions.cs

@@ -1,7 +1,3 @@
-using Microsoft.Extensions.Logging;
-
-#nullable enable
-
 namespace VariantAndAzureMonitorDemo
 {
     public static class LoggerExtensions
@@ -15,6 +11,7 @@ public static class LoggerExtensions
             LogLevel.Information,
             new EventId(1, "microsoft.custom_event.name"),
             "{microsoft.custom_event.name} {success} {amount}");
+
         public static void LogVote(this ILogger logger, int rating)
         {
             _vote(logger, "Vote", rating, null);

tests/Tests.FeatureManagement.Telemetry.AzureMonitor/ProcessorOrderTests.cs

@@ -24,21 +24,21 @@ public void TargetingProcessor_RunsBefore_Exporter()
             // 1. User adds OpenTelemetry and Exporter
             // Note: In a real app, UseAzureMonitor() would add the exporter.
             // We simulate this by adding a processor that acts as an exporter.
+            var exportProcessor = new SimpleActivityExportProcessor(exporter);
             services.AddOpenTelemetry()
                 .WithTracing(builder => builder
                     .AddSource("TestTracer")
-                    .AddProcessor(new SimpleActivityExportProcessor(exporter)));
+                    .AddProcessor(exportProcessor));
 
             // 2. User adds FeatureManagement and AzureMonitorTelemetry
             services.AddFeatureManagement()
                 .AddAzureMonitorTelemetry();
 
             using var serviceProvider = services.BuildServiceProvider();
             var tracerProvider = serviceProvider.GetRequiredService<TracerProvider>();
-            var tracer = tracerProvider.GetTracer("TestTracer");
 
             // 3. Start Activity with Baggage
-            var source = new ActivitySource("TestTracer");
+            using var source = new ActivitySource("TestTracer");
             using (var listener = new ActivityListener
             {
                 ShouldListenTo = _ => true,
@@ -61,6 +61,8 @@ public void TargetingProcessor_RunsBefore_Exporter()
 
             // If TargetingProcessor ran first, the tag should be present.
             Assert.Contains(tags, t => t.Key == "TargetingId" && t.Value == "User123");
+
+            exportProcessor.Dispose();
         }
 
         private class InMemoryExporter : BaseExporter<Activity>