Skip to content

Conversation

@hannahkm
Copy link
Contributor

@hannahkm hannahkm commented Dec 9, 2025

WIP

Tested w a demo service that wraps net/http handlers and forces them to generate a span. One has an ignore directive. Only the handler without the ignore directive produces a span:
image

Tested on echo as well:
image

@github-actions github-actions bot added the conventional-commit/feat Feature work label Dec 9, 2025
@codecov
Copy link

codecov bot commented Dec 10, 2025

Codecov Report

❌ Patch coverage is 77.77778% with 4 lines in your changes missing coverage. Please review.
✅ Project coverage is 69.58%. Comparing base (e061d12) to head (9dcb532).
⚠️ Report is 47 commits behind head on main.

Files with missing lines Patch % Lines
internal/injector/aspect/join/function.go 77.77% 2 Missing and 2 partials ⚠️
Additional details and impacted files
@@            Coverage Diff             @@
##             main     #756      +/-   ##
==========================================
+ Coverage   65.72%   69.58%   +3.86%     
==========================================
  Files         113      116       +3     
  Lines        7926     6907    -1019     
==========================================
- Hits         5209     4806     -403     
+ Misses       2192     1552     -640     
- Partials      525      549      +24     
Components Coverage Δ
Generators 83.23% <ø> (+2.98%) ⬆️
Instruments ∅ <ø> (∅)
Go Driver 77.08% <71.15%> (+1.27%) ⬆️
Toolexec Driver 74.73% <100.00%> (+7.20%) ⬆️
Aspects 76.73% <76.50%> (+4.82%) ⬆️
Injector 76.98% <77.10%> (+4.18%) ⬆️
Job Server 67.50% <55.55%> (+1.58%) ⬆️
Other 69.58% <66.61%> (+3.86%) ⬆️
Files with missing lines Coverage Δ
internal/injector/aspect/join/function.go 72.70% <77.77%> (+6.28%) ⬆️

... and 107 files with indirect coverage changes

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

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

Labels

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants