Closed CodeBlanch closed 1 week ago
OTLP, Zipkin, & Prometheus exporters all expose shapes which are (essentially):
public static ProviderBuilder AddExporter(this ProviderBuilder builder) {}; public static ProviderBuilder AddExporter(this ProviderBuilder builder, Action<ExporterOptions> configure) {} public static ProviderBuilder AddExporter(this ProviderBuilder builder, string? name, Action<ExporterOptions>? configure) {}
The second one sometimes has Action<ExporterOptions>? configure this PR is dropping that ? in the signature. null is still allowed, it just isn't encouraged/promoted. For more thoughts/details see: https://github.com/open-telemetry/opentelemetry-dotnet-contrib/pull/2067#discussion_r1758241013
Action<ExporterOptions>? configure
?
null
All modified and coverable lines are covered by tests :white_check_mark:
Project coverage is 86.21%. Comparing base (6250307) to head (5fd4faa). Report is 315 commits behind head on main.
6250307
5fd4faa
Changes
Details
OTLP, Zipkin, & Prometheus exporters all expose shapes which are (essentially):
The second one sometimes has
Action<ExporterOptions>? configure
this PR is dropping that?
in the signature.null
is still allowed, it just isn't encouraged/promoted. For more thoughts/details see: https://github.com/open-telemetry/opentelemetry-dotnet-contrib/pull/2067#discussion_r1758241013Merge requirement checklist