Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Jump to bottom

Allow span type along with SpanContext everywhere #58

Closed
mayurkale22 opened this issue Jun 25, 2019 · 1 comment
Closed

Allow span type along with SpanContext everywhere #58

mayurkale22 opened this issue Jun 25, 2019 · 1 comment
Assignees
@mayurkale22
Labels
Discussion Issue or PR that needs/is extended discussion.

Comments

@mayurkale22
Copy link
Member

This avoids the caller having to always call span.context() if they are referencing a span.

Originates from: #55 (comment) and #31 (comment)
@mayurkale22 mayurkale22 added Discussion Issue or PR that needs/is extended discussion. API labels Jun 25, 2019
@mayurkale22 mayurkale22 added this to the API complete milestone Jun 25, 2019
@mayurkale22 mayurkale22 mentioned this issue Jun 25, 2019
Merged
@mayurkale22 mayurkale22 added the up-for-grabs Good for taking. Extra help will be provided by maintainers label Aug 19, 2019
@mayurkale22 mayurkale22 modified the milestones: API complete, Alpha v0.3 Oct 11, 2019
@mayurkale22 mayurkale22 added the good first issue Good for newcomers label Oct 22, 2019
@mayurkale22 mayurkale22 self-assigned this Nov 20, 2019
@mayurkale22 mayurkale22 removed good first issue Good for newcomers up-for-grabs Good for taking. Extra help will be provided by maintainers labels Nov 22, 2019
@mayurkale22 mayurkale22 modified the milestones: Alpha v0.3, Alpha v0.4 Dec 23, 2019
@mayurkale22 mayurkale22 removed this from the Alpha v0.4 milestone Feb 12, 2020
@mayurkale22
Copy link
Member Author

Obsolete, closing now.

dyladan added a commit to dyladan/opentelemetry-js that referenced this issue Sep 9, 2022
@dyladan
chore: use spancontext for link (open-telemetry#58)
fa99923
dyladan added a commit to dyladan/opentelemetry-js that referenced this issue Sep 9, 2022
@dyladan
chore: use spancontext for link (open-telemetry#58)
f5d8aaa
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees

@mayurkale22 mayurkale22

Labels
Discussion Issue or PR that needs/is extended discussion.
Projects
None yet
Milestone
No milestone
Development

No branches or pull requests
1 participant
@mayurkale22