Skip to content
This repository has been archived by the owner on Oct 3, 2023. It is now read-only.

Cleanup: remove redundant types in JSDoc #510

Merged
merged 1 commit into from
May 9, 2019

Conversation

mayurkale22
Copy link
Member

No description provided.

Copy link
Contributor

@draffensperger draffensperger 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 fixing these!

@codecov-io
Copy link

Codecov Report

Merging #510 into master will increase coverage by 0.05%.
The diff coverage is n/a.

Impacted file tree graph

@@            Coverage Diff             @@
##           master     #510      +/-   ##
==========================================
+ Coverage   95.03%   95.09%   +0.05%     
==========================================
  Files         145      145              
  Lines       10214    10350     +136     
  Branches      893      885       -8     
==========================================
+ Hits         9707     9842     +135     
- Misses        507      508       +1
Impacted Files Coverage Δ
src/trace/propagation/noop-propagation.ts 40% <0%> (-10%) ⬇️
src/common/noop-logger.ts 11.11% <0%> (-5.56%) ⬇️
src/common-utils.ts 95.83% <0%> (-4.17%) ⬇️
src/internal/string-utils.ts 100% <0%> (ø) ⬆️
src/http-stats.ts 100% <0%> (ø) ⬆️
test/test-tag-map.ts 100% <0%> (ø) ⬆️
src/binary-format.ts 100% <0%> (ø) ⬆️
src/metrics/gauges/gauge.ts 100% <0%> (ø) ⬆️
test/test-metric-producer-manager.ts 100% <0%> (ø) ⬆️
test/test-exporter-buffer.ts 100% <0%> (ø) ⬆️
... and 31 more

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 5db22c9...0d1a312. Read the comment docs.

@mayurkale22 mayurkale22 added this to the Release 0.0.12 milestone May 9, 2019
@mayurkale22 mayurkale22 merged commit 111223e into census-instrumentation:master May 9, 2019
@mayurkale22 mayurkale22 deleted the cleanup branch May 9, 2019 18:24
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants