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

Added 'w3c' to valid JAEGER_PROPAGATION options #736

Merged
merged 1 commit into from
Aug 17, 2020

Conversation

chgl
Copy link
Contributor

@chgl chgl commented Aug 17, 2020

Which problem is this PR solving?

Resolves #733

Short description of the changes

  • added w3c option to docs

Signed-off-by: chgl <chgl@users.noreply.github.com>
@codecov
Copy link

codecov bot commented Aug 17, 2020

Codecov Report

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

Impacted file tree graph

@@             Coverage Diff              @@
##             master     #736      +/-   ##
============================================
- Coverage     89.08%   88.77%   -0.32%     
+ Complexity      601      598       -3     
============================================
  Files            73       73              
  Lines          2253     2253              
  Branches        292      292              
============================================
- Hits           2007     2000       -7     
- Misses          155      161       +6     
- Partials         91       92       +1     
Impacted Files Coverage Δ Complexity Δ
...java/io/jaegertracing/zipkin/ZipkinV2Reporter.java 71.42% <0.00%> (-28.58%) 2.00% <0.00%> (-1.00%)
...rtracing/internal/reporters/CompositeReporter.java 71.42% <0.00%> (-28.58%) 6.00% <0.00%> (-1.00%)
...gertracing/internal/reporters/LoggingReporter.java 81.81% <0.00%> (-9.10%) 4.00% <0.00%> (-1.00%)

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 a3bf0e3...29f9b24. 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 free to subscribe to this conversation on GitHub. Already have an account? #.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

Does the JAEGER_PROPAGATION env var support the w3c setting?
2 participants