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

Update uncompressedSize and compressedSize for the exporters #347

Merged

Conversation

mayurkale22
Copy link
Member

This PR is based on the discussion here: census-instrumentation/opencensus-website#580.

@codecov-io
Copy link

Codecov Report

Merging #347 into master will increase coverage by <.01%.
The diff coverage is n/a.

Impacted file tree graph

@@            Coverage Diff             @@
##           master     #347      +/-   ##
==========================================
+ Coverage   95.14%   95.15%   +<.01%     
==========================================
  Files         122      122              
  Lines        8474     8478       +4     
  Branches      750      754       +4     
==========================================
+ Hits         8063     8067       +4     
  Misses        411      411
Impacted Files Coverage Δ
test/test-ocagent.ts 91.74% <0%> (ø) ⬆️
test/test-stackdriver-cloudtrace-utils.ts 100% <0%> (ø) ⬆️
src/adapters.ts 97% <0%> (+0.06%) ⬆️
src/stackdriver-cloudtrace-utils.ts 97.26% <0%> (+0.07%) ⬆️

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 3638ca5...c44eaf2. Read the comment docs.

@draffensperger
Copy link
Contributor

Thanks for doing this!

@mayurkale22 mayurkale22 merged commit 0605a67 into census-instrumentation:master Feb 18, 2019
@mayurkale22 mayurkale22 deleted the addMessageEvent branch February 18, 2019 19:48
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.

None yet

4 participants