Skip to content

Commit

Permalink
[AzureMonitorExporter] cleanup csproj. add missing XML comments (Azur…
Browse files Browse the repository at this point in the history
…e#32544)

* cleanup csproj. resolve missing XML comments

* Update sdk/monitor/Azure.Monitor.OpenTelemetry.Exporter/src/AzureMonitorExporterOptions.cs

Co-authored-by: Rajkumar Rangaraj <rajrang@microsoft.com>

Co-authored-by: Rajkumar Rangaraj <rajrang@microsoft.com>
  • Loading branch information
TimothyMothra and rajkumar-rangaraj authored Nov 22, 2022
1 parent 7adc746 commit 2627a3c
Show file tree
Hide file tree
Showing 2 changed files with 10 additions and 1 deletion.
Original file line number Diff line number Diff line change
Expand Up @@ -27,7 +27,7 @@
<PropertyGroup>
<!-- TODO: As part of release process, this PropertyGroup can be removed once RP is approved for namespace
https://github.com/Azure/azure-sdk-for-net/issues/17155 -->
<NoWarn>$(NoWarn);AZC0001;CS0108;CS1591</NoWarn>
<NoWarn>$(NoWarn);AZC0001</NoWarn>
</PropertyGroup>

</Project>
Original file line number Diff line number Diff line change
Expand Up @@ -39,6 +39,9 @@ public AzureMonitorExporterOptions(ServiceVersion version = LatestVersion)
this.Version = version;
}

/// <summary>
/// The versions of Azure Monitor supported by this client library.
/// </summary>
[SuppressMessage("Naming", "CA1707:Identifiers should not contain underscores", Justification = "This naming format is defined in the Azure SDK Design Guidelines.")]
public enum ServiceVersion
{
Expand All @@ -48,8 +51,14 @@ public enum ServiceVersion
V2020_09_15_Preview = 1,
}

/// <summary>
/// Override the default directory for offline storage.
/// </summary>
public string StorageDirectory { get; set; }

/// <summary>
/// Disable offline storage.
/// </summary>
public bool DisableOfflineStorage { get; set; }
}
}

0 comments on commit 2627a3c

Please sign in to comment.