Skip to content

Increase use of NotNullWhen polyfill #6090

Merged
rajkumar-rangaraj merged 8 commits intoopen-telemetry:mainfrom
SimonCropp:leverage-more-NotNullWhen-polyfill-via-an-InternalsVIsibleTo
Feb 11, 2025
Merged

Increase use of NotNullWhen polyfill #6090
rajkumar-rangaraj merged 8 commits intoopen-telemetry:mainfrom
SimonCropp:leverage-more-NotNullWhen-polyfill-via-an-InternalsVIsibleTo

Conversation

@SimonCropp
Copy link
Contributor

Fixes #
Design discussion issue #

Changes

also allows us to remove some redundant ! null suppressions

Merge requirement checklist

  • CONTRIBUTING guidelines followed (license requirements, nullable enabled, static analysis, etc.)
  • Unit tests added/updated
  • Appropriate CHANGELOG.md files updated for non-trivial changes
  • Changes in public API reviewed (if applicable)

@SimonCropp SimonCropp requested a review from a team as a code owner January 23, 2025 05:04
@github-actions github-actions bot added pkg:OpenTelemetry.Exporter.OpenTelemetryProtocol Issues related to OpenTelemetry.Exporter.OpenTelemetryProtocol NuGet package pkg:OpenTelemetry.Exporter.Zipkin Issues related to OpenTelemetry.Exporter.Zipkin NuGet package pkg:OpenTelemetry Issues related to OpenTelemetry NuGet package pkg:OpenTelemetry.Api Issues related to OpenTelemetry.Api NuGet package labels Jan 23, 2025
@SimonCropp SimonCropp changed the title leverage more NotNullWhen polyfill via an InternalsVIsibleTo Increase use of NotNullWhen polyfill Jan 23, 2025
@TimothyMothra
Copy link

LGTM. You have conflicts with main and will need to update your branch

@SimonCropp
Copy link
Contributor Author

conflicts resolved

@github-actions
Copy link
Contributor

github-actions bot commented Feb 3, 2025

This PR was marked stale due to lack of activity and will be closed in 7 days. Commenting or pushing will instruct the bot to automatically remove the label. This bot runs once per day.

@github-actions github-actions bot added the Stale Issues and pull requests which have been flagged for closing due to inactivity label Feb 3, 2025
@codecov
Copy link

codecov bot commented Feb 3, 2025

Codecov Report

All modified and coverable lines are covered by tests ✅

Project coverage is 86.36%. Comparing base (720d610) to head (0fec244).
Report is 1 commits behind head on main.

Additional details and impacted files

Impacted file tree graph

@@            Coverage Diff             @@
##             main    #6090      +/-   ##
==========================================
- Coverage   86.36%   86.36%   -0.01%     
==========================================
  Files         257      257              
  Lines       11584    11579       -5     
==========================================
- Hits        10005    10000       -5     
  Misses       1579     1579              
Files with missing lines Coverage Δ
...etryProtocol/Implementation/ExperimentalOptions.cs 88.23% <ø> (-1.24%) ⬇️
...Telemetry.Exporter.Zipkin/ZipkinExporterOptions.cs 100.00% <ø> (ø)
src/OpenTelemetry/Logs/LoggerProviderSdk.cs 92.55% <ø> (ø)
...OpenTelemetry/Resources/OtelEnvResourceDetector.cs 100.00% <ø> (ø)
...lemetry/Resources/OtelServiceNameEnvVarDetector.cs 100.00% <ø> (ø)

@TimothyMothra TimothyMothra removed the Stale Issues and pull requests which have been flagged for closing due to inactivity label Feb 6, 2025
@rajkumar-rangaraj rajkumar-rangaraj merged commit 41e1a59 into open-telemetry:main Feb 11, 2025
49 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

pkg:OpenTelemetry.Api Issues related to OpenTelemetry.Api NuGet package pkg:OpenTelemetry.Exporter.OpenTelemetryProtocol Issues related to OpenTelemetry.Exporter.OpenTelemetryProtocol NuGet package pkg:OpenTelemetry.Exporter.Zipkin Issues related to OpenTelemetry.Exporter.Zipkin NuGet package pkg:OpenTelemetry Issues related to OpenTelemetry NuGet package ready to merge

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants