Skip to content

Conversation

@codeboten
Copy link
Contributor

The underlying types were all the same, so this wasn't caught by the casting at the end of each func. Worth fixing to remove confusion.

Didn't add a changelog since the code in question isn't out yet.

The underlying types were all the same, so this wasn't caught by the casting at the end of each func. Worth fixing to remove confusion.

Signed-off-by: alex boten <[email protected]>
@codeboten codeboten requested review from a team and pellared as code owners November 14, 2025 23:18
@codecov
Copy link

codecov bot commented Nov 17, 2025

Codecov Report

✅ All modified and coverable lines are covered by tests.
✅ Project coverage is 81.8%. Comparing base (183643f) to head (c936c16).
⚠️ Report is 1 commits behind head on main.

Additional details and impacted files

Impacted file tree graph

@@          Coverage Diff          @@
##            main   #8181   +/-   ##
=====================================
  Coverage   81.8%   81.8%           
=====================================
  Files        194     194           
  Lines      13377   13377           
=====================================
  Hits       10951   10951           
  Misses      2029    2029           
  Partials     397     397           
Files with missing lines Coverage Δ
otelconf/config_json.go 92.9% <100.0%> (ø)
🚀 New features to boost your workflow:
  • ❄️ Test Analytics: Detect flaky tests, report on failures, and find test suite problems.

@dashpole dashpole added the Skip Changelog Allow PR to succeed without requiring an addition to the CHANGELOG label Nov 17, 2025
@dmathieu dmathieu merged commit 0f4af38 into open-telemetry:main Nov 18, 2025
28 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

Skip Changelog Allow PR to succeed without requiring an addition to the CHANGELOG

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants