Skip to content

Conversation

@mayurkale22
Copy link
Member

Which problem is this PR solving?

@codecov-io
Copy link

codecov-io commented Sep 19, 2019

Codecov Report

Merging #289 into master will increase coverage by 0.03%.
The diff coverage is n/a.

@@            Coverage Diff             @@
##           master     #289      +/-   ##
==========================================
+ Coverage   98.86%   98.89%   +0.03%     
==========================================
  Files          78       78              
  Lines        3247     3163      -84     
  Branches      211      213       +2     
==========================================
- Hits         3210     3128      -82     
+ Misses         37       35       -2
Impacted Files Coverage Δ
.../opentelemetry-exporter-jaeger/test/jaeger.test.ts 100% <0%> (ø) ⬆️
...ges/opentelemetry-exporter-jaeger/src/transform.ts 100% <0%> (ø) ⬆️
...entelemetry-exporter-jaeger/test/transform.test.ts 100% <0%> (ø) ⬆️
...ckages/opentelemetry-core/test/common/time.test.ts 100% <0%> (ø) ⬆️
packages/opentelemetry-core/src/common/time.ts 98.21% <0%> (+1.6%) ⬆️
...ckages/opentelemetry-exporter-jaeger/src/jaeger.ts 92% <0%> (+2.34%) ⬆️

@mayurkale22 mayurkale22 requested a review from hekike September 19, 2019 21:31
@mayurkale22 mayurkale22 merged commit a1f5f43 into open-telemetry:master Sep 20, 2019
@mayurkale22 mayurkale22 deleted the zipkin_main branch September 20, 2019 04:30
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

fix(exporter-zipkin): add main property - entry point to the module

5 participants