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

chore: use spancontext for link #58

Merged
merged 3 commits into from
May 5, 2021

Conversation

dyladan
Copy link
Member

@dyladan dyladan commented May 3, 2021

Fixes #57

@codecov
Copy link

codecov bot commented May 3, 2021

Codecov Report

Merging #58 (e79783c) into main (c595a59) will decrease coverage by 0.01%.
The diff coverage is n/a.

Impacted file tree graph

@@            Coverage Diff             @@
##             main      #58      +/-   ##
==========================================
- Coverage   94.72%   94.71%   -0.02%     
==========================================
  Files          40       40              
  Lines         512      511       -1     
  Branches       80       80              
==========================================
- Hits          485      484       -1     
  Misses         27       27              
Impacted Files Coverage Δ
src/index.ts 100.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 c595a59...e79783c. Read the comment docs.

src/trace/link.ts Outdated Show resolved Hide resolved
@dyladan
Copy link
Member Author

dyladan commented May 3, 2021

@Flarna Flarna added the breaking label May 4, 2021
Copy link
Member

@obecny obecny left a comment

Choose a reason for hiding this comment

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

lgtm

@dyladan dyladan merged commit 9b3f9c7 into open-telemetry:main May 5, 2021
@dyladan dyladan deleted the span-context-link branch May 5, 2021 17:22
@dyladan dyladan mentioned this pull request May 17, 2021
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.

Use whole spancontext for link
6 participants