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

Add Trace Params #311#315

Closed
vigneshtdev wants to merge 7 commits intocensus-instrumentation:masterfrom
vigneshtdev:master
Closed

Add Trace Params #311#315
vigneshtdev wants to merge 7 commits intocensus-instrumentation:masterfrom
vigneshtdev:master

Conversation

@vigneshtdev
Copy link
Copy Markdown
Contributor

No description provided.

@codecov-io
Copy link
Copy Markdown

Codecov Report

Merging #315 into master will decrease coverage by 0.18%.
The diff coverage is n/a.

Impacted file tree graph

@@            Coverage Diff             @@
##           master     #315      +/-   ##
==========================================
- Coverage    94.9%   94.72%   -0.19%     
==========================================
  Files         118      114       -4     
  Lines        8030     7655     -375     
  Branches      717      701      -16     
==========================================
- Hits         7621     7251     -370     
+ Misses        409      404       -5
Impacted Files Coverage Δ
src/trace/sampler/sampler.ts 92% <0%> (-8%) ⬇️
src/trace/model/tracer.ts 82.45% <0%> (-2.01%) ⬇️
...zpages-frontend/page-handlers/rpcz.page-handler.ts 98.92% <0%> (-1.08%) ⬇️
src/prometheus-stats.ts 94.25% <0%> (-0.49%) ⬇️
src/stats/recorder.ts 97.43% <0%> (-0.3%) ⬇️
src/stats/stats.ts 98.24% <0%> (-0.09%) ⬇️
src/stackdriver-stats-utils.ts 99.01% <0%> (-0.01%) ⬇️
test/test-zpages.ts 99.27% <0%> (-0.01%) ⬇️
src/stats/types.ts 100% <0%> (ø) ⬆️
test/test-sampler.ts 100% <0%> (ø) ⬆️
... and 18 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 9414faf...9c97a0c. Read the comment docs.

@vigneshtdev
Copy link
Copy Markdown
Contributor Author

Contd. from #311
Sorry about creating a different PR. i made changes in the wrong branch.

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.

3 participants