Skip to content
This repository has been archived by the owner on Sep 13, 2022. It is now read-only.

Add missed contextKey option to initTracer #308

Merged

Conversation

yepninja
Copy link
Contributor

@yepninja yepninja commented Nov 1, 2018

Resolves #254

Signed-off-by: Yevgeny Petukhov <petukhov.yevgeny@gmail.com>
@codecov
Copy link

codecov bot commented Nov 1, 2018

Codecov Report

Merging #308 into master will not change coverage.
The diff coverage is n/a.

Impacted file tree graph

@@           Coverage Diff           @@
##           master     #308   +/-   ##
=======================================
  Coverage   98.93%   98.93%           
=======================================
  Files          43       43           
  Lines        1697     1697           
  Branches      322      322           
=======================================
  Hits         1679     1679           
  Misses         18       18
Impacted Files Coverage Δ
src/configuration.js 98.94% <ø> (ø) ⬆️
src/tracer.js 100% <ø> (ø) ⬆️

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 cde1de1...e5666ff. Read the comment docs.

Copy link
Member

@yurishkuro yurishkuro left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Thanks for the fix!

@yurishkuro yurishkuro merged commit 24c950d into jaegertracing:master Nov 1, 2018
@yurishkuro
Copy link
Member

@yepninja if your company is using Jaeger, please consider commenting on jaegertracing/jaeger#207

Iuriy-Budnikov pushed a commit to agile-pm/jaeger-client-node that referenced this pull request Sep 25, 2021
closes jaegertracing#307

Signed-off-by: Olivier Albertini <olivier.albertini@montreal.ca>
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

2 participants