Skip to content

Add private constructors for stateless marshalers - #6434

Merged
jack-berg merged 1 commit into
open-telemetry:mainfrom
laurit:marshaler-private-ctor
May 7, 2024
Merged

Add private constructors for stateless marshalers#6434
jack-berg merged 1 commit into
open-telemetry:mainfrom
laurit:marshaler-private-ctor

Conversation

@laurit

@laurit laurit commented May 7, 2024

Copy link
Copy Markdown
Contributor

Skipped the ones that are modified in #6429

@laurit
laurit requested a review from a team May 7, 2024 07:17
@codecov

codecov Bot commented May 7, 2024

Copy link
Copy Markdown

Codecov Report

All modified and coverable lines are covered by tests ✅

Project coverage is 90.85%. Comparing base (36bc703) to head (47fb68d).

Additional details and impacted files
@@             Coverage Diff              @@
##               main    #6434      +/-   ##
============================================
- Coverage     90.86%   90.85%   -0.01%     
+ Complexity     6030     6020      -10     
============================================
  Files           651      651              
  Lines         17728    17718      -10     
  Branches       1777     1777              
============================================
- Hits          16108    16098      -10     
  Misses         1105     1105              
  Partials        515      515              

☔ View full report in Codecov by Sentry.
📢 Have feedback on the report? Share it here.

@jack-berg jack-berg left a comment

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Thanks for following up on this!

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants