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.

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

chore: creating new metric kind #1145

Merged
merged 4 commits into from
Jun 6, 2020

Conversation

obecny
Copy link
Member

@obecny obecny commented Jun 4, 2020

Which problem is this PR solving?

  • this is to create new kind of metric without deleting old one (observer)
    this can be done later when PR for new metris will be merged

https://github.com/open-telemetry/opentelemetry-specification/blob/master/specification/metrics/api.md#interpretation

@obecny obecny mentioned this pull request Jun 4, 2020
@codecov
Copy link

codecov bot commented Jun 4, 2020

Codecov Report

Merging #1145 into master will increase coverage by 0.00%.
The diff coverage is 100.00%.

@@           Coverage Diff           @@
##           master    #1145   +/-   ##
=======================================
  Coverage   92.30%   92.31%           
=======================================
  Files         116      116           
  Lines        3393     3397    +4     
  Branches      685      685           
=======================================
+ Hits         3132     3136    +4     
  Misses        261      261           
Impacted Files Coverage Δ
packages/opentelemetry-metrics/src/export/types.ts 100.00% <100.00%> (ø)

@obecny obecny self-assigned this Jun 4, 2020
@obecny obecny added the size/XS label Jun 5, 2020
@linux-foundation-easycla
Copy link

linux-foundation-easycla bot commented Jun 5, 2020

CLA Check
The committers are authorized under a signed CLA.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

6 participants