Closed duncanista closed 10 months ago
Merging #432 (4a255dc) into main (35295b3) will increase coverage by
0.81%
. Report is 81 commits behind head on main. The diff coverage is82.91%
.:exclamation: Current head 4a255dc differs from pull request most recent head 329db45. Consider uploading reports for the commit 329db45 to get more accurate results
@@ Coverage Diff @@
## main #432 +/- ##
==========================================
+ Coverage 80.35% 81.16% +0.81%
==========================================
Files 36 50 +14
Lines 1761 2039 +278
Branches 410 466 +56
==========================================
+ Hits 1415 1655 +240
- Misses 294 323 +29
- Partials 52 61 +9
Files | Coverage Δ | |
---|---|---|
src/constants.ts | 100.00% <100.00%> (ø) |
|
src/metrics/enhanced-metrics.ts | 91.89% <100.00%> (-0.61%) |
:arrow_down: |
src/metrics/listener.ts | 85.18% <100.00%> (-0.19%) |
:arrow_down: |
src/runtime/index.ts | 100.00% <100.00%> (+100.00%) |
:arrow_up: |
src/runtime/user-function.ts | 7.69% <ø> (+7.69%) |
:arrow_up: |
src/trace/constants.ts | 100.00% <100.00%> (ø) |
|
src/trace/context/extractors/app-sync.ts | 100.00% <100.00%> (ø) |
|
src/trace/context/extractors/step-function.ts | 100.00% <100.00%> (ø) |
|
src/trace/trace-context-service.ts | 83.33% <100.00%> (-0.54%) |
:arrow_down: |
src/trace/tracer-wrapper.ts | 80.48% <100.00%> (ø) |
|
... and 23 more |
... and 1 file with indirect coverage changes
:mega: Codecov offers a browser extension for seamless coverage viewing on GitHub. Try it in Chrome or Firefox today!
What does this PR do?
Re organizes our Trace Context extraction from one file to multiple separated files.
Motivation
For #429, it's ideal to have this PR as predecessor in order to ease review.
Testing Guidelines
n/a
Additional Notes
Types of Changes
Check all that apply