Skip to content

feat(stdouttrace): generate counter implementation via templates#7231

Merged
MrAlias merged 5 commits intoopen-telemetry:mainfrom
yumosx:counter-tmpl
Aug 24, 2025
Merged

feat(stdouttrace): generate counter implementation via templates#7231
MrAlias merged 5 commits intoopen-telemetry:mainfrom
yumosx:counter-tmpl

Conversation

@yumosx
Copy link
Copy Markdown
Member

@yumosx yumosx commented Aug 21, 2025

Use templates to generate the counter implementation for stdouttrace, so it can be easily reused in other modules.

@codecov
Copy link
Copy Markdown

codecov Bot commented Aug 21, 2025

Codecov Report

✅ All modified and coverable lines are covered by tests.
✅ Project coverage is 83.0%. Comparing base (70e8929) to head (8b2d5e7).
⚠️ Report is 1 commits behind head on main.

Additional details and impacted files

Impacted file tree graph

@@           Coverage Diff           @@
##            main   #7231     +/-   ##
=======================================
- Coverage   83.0%   83.0%   -0.1%     
=======================================
  Files        265     265             
  Lines      24661   24661             
=======================================
- Hits       20476   20473      -3     
- Misses      3808    3811      +3     
  Partials     377     377             
Files with missing lines Coverage Δ
...ers/stdout/stdouttrace/internal/counter/counter.go 100.0% <ø> (ø)

... and 2 files with indirect coverage changes

🚀 New features to boost your workflow:
  • ❄️ Test Analytics: Detect flaky tests, report on failures, and find test suite problems.

@MrAlias MrAlias added the Skip Changelog PRs that do not require a CHANGELOG.md entry label Aug 21, 2025
@MrAlias MrAlias merged commit c2a6172 into open-telemetry:main Aug 24, 2025
37 checks passed
@MrAlias MrAlias added this to the v1.38.0 milestone Aug 26, 2025
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

Skip Changelog PRs that do not require a CHANGELOG.md entry

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants