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

chore: function overloads implementation of startActiveSpan in noop t… #81

Merged
merged 9 commits into from Jun 1, 2021

Conversation

naseemkullah
Copy link
Member

…racer

Signed-off-by: naseemkullah naseem@transit.app

…racer

Signed-off-by: naseemkullah <naseem@transit.app>
@codecov
Copy link

codecov bot commented May 21, 2021

Codecov Report

Merging #81 (5217cc8) into main (1a35772) will increase coverage by 0.10%.
The diff coverage is 100.00%.

Impacted file tree graph

@@            Coverage Diff             @@
##             main      #81      +/-   ##
==========================================
+ Coverage   94.72%   94.83%   +0.10%     
==========================================
  Files          42       42              
  Lines         569      561       -8     
  Branches       96       88       -8     
==========================================
- Hits          539      532       -7     
+ Misses         30       29       -1     
Impacted Files Coverage Δ
src/trace/NoopTracer.ts 97.36% <100.00%> (+1.71%) ⬆️

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 1a35772...5217cc8. Read the comment docs.

naseemkullah added 3 commits May 21, 2021 20:39
Signed-off-by: naseemkullah <naseem@transit.app>
Signed-off-by: naseemkullah <naseem@transit.app>
Signed-off-by: naseemkullah <naseem@transit.app>
src/trace/NoopTracer.ts Outdated Show resolved Hide resolved
src/trace/NoopTracer.ts Outdated Show resolved Hide resolved
Naseem and others added 2 commits May 24, 2021 11:59
Co-authored-by: Daniel Dyla <dyladan@users.noreply.github.com>
Co-authored-by: Daniel Dyla <dyladan@users.noreply.github.com>
@naseemkullah naseemkullah requested a review from dyladan May 24, 2021 17:09
@dyladan dyladan added the enhancement New feature or request label May 27, 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

@naseemkullah
Copy link
Member Author

The base branch restricts merging to authorized users.

Could someone with authority please merge?

@obecny
Copy link
Member

obecny commented May 31, 2021

The base branch restricts merging to authorized users.

Could someone with authority please merge?

on api repo we want all maintainers to approve.

@dyladan dyladan merged commit 35224c7 into open-telemetry:main Jun 1, 2021
@naseemkullah naseemkullah deleted the tracer-start-active-span branch June 1, 2021 20:44
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
enhancement New feature or request
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

5 participants