Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

[API] Propagators: do not overwrite the active span with a default invalid span #2511

Merged

Merge branch 'main' into fixCompositePropagator

cf7a4fd
Select commit
Loading
Failed to load commit list.
Merged

[API] Propagators: do not overwrite the active span with a default invalid span #2511

Merge branch 'main' into fixCompositePropagator
cf7a4fd
Select commit
Loading
Failed to load commit list.
Codecov / codecov/project succeeded Jan 27, 2024 in 0s

87.12% (+0.05%) compared to 2320636

View this Pull Request on Codecov

87.12% (+0.05%) compared to 2320636

Details

Codecov Report

All modified and coverable lines are covered by tests ✅

Comparison is base (2320636) 87.08% compared to head (cf7a4fd) 87.12%.

Additional details and impacted files

Impacted file tree graph

@@            Coverage Diff             @@
##             main    #2511      +/-   ##
==========================================
+ Coverage   87.08%   87.12%   +0.05%     
==========================================
  Files         200      200              
  Lines        6103     6109       +6     
==========================================
+ Hits         5314     5322       +8     
+ Misses        789      787       -2     
Files Coverage Δ
...de/opentelemetry/trace/propagation/b3_propagator.h 93.66% <100.00%> (+0.21%) ⬆️
...entelemetry/trace/propagation/http_trace_context.h 95.59% <100.00%> (+1.65%) ⬆️
...i/include/opentelemetry/trace/propagation/jaeger.h 97.78% <100.00%> (+0.11%) ⬆️

... and 1 file with indirect coverage changes