Skip to content
This repository has been archived by the owner on Oct 3, 2023. It is now read-only.

Cleanup type assertion #525

Merged
merged 2 commits into from
May 15, 2019

Conversation

mayurkale22
Copy link
Member

No description provided.

@codecov-io
Copy link

Codecov Report

Merging #525 into master will decrease coverage by 0.04%.
The diff coverage is n/a.

Impacted file tree graph

@@            Coverage Diff             @@
##           master     #525      +/-   ##
==========================================
- Coverage   95.27%   95.22%   -0.05%     
==========================================
  Files         147      147              
  Lines       10808    10807       -1     
  Branches      907      916       +9     
==========================================
- Hits        10297    10291       -6     
- Misses        511      516       +5
Impacted Files Coverage Δ
src/common-utils.ts 95.83% <0%> (-4.17%) ⬇️
test/test-zipkin.ts 97.82% <0%> (-2.18%) ⬇️
test/test-jaeger-format.ts 95.38% <0%> (-1.8%) ⬇️
test/test-stackdriver-cloudtrace.ts 98.96% <0%> (-1.04%) ⬇️
test/test-jaeger.ts 98.88% <0%> (-0.39%) ⬇️
src/tracecontext-format.ts 96.42% <0%> (-0.19%) ⬇️
src/trace/model/span.ts 97.91% <0%> (-0.05%) ⬇️
test/test-tracecontext-format.ts 99.06% <0%> (-0.04%) ⬇️
src/http-stats.ts 100% <0%> (ø) ⬆️
src/grpc-stats/client-stats.ts 100% <0%> (ø) ⬆️
... and 7 more

Continue to review full report at Codecov.

Legend - Click here to learn more
Δ = absolute <relative> (impact), ø = not affected, ? = missing data
Powered by Codecov. Last update 1a77de1...ee63132. Read the comment docs.

@mayurkale22
Copy link
Member Author

@draffensperger thanks for the reviews.

@mayurkale22 mayurkale22 merged commit 0fcb0cb into census-instrumentation:master May 15, 2019
@mayurkale22 mayurkale22 deleted the cleanup branch May 15, 2019 17:12
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

4 participants