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

feat: send one sample event, response per label set in the configured duration #5298

Open
wants to merge 4 commits into
base: feat.reports-bactching
Choose a base branch
from

fix: minor

bc0acdd
Select commit
Loading
Failed to load commit list.
Open

feat: send one sample event, response per label set in the configured duration #5298

fix: minor
bc0acdd
Select commit
Loading
Failed to load commit list.
Codecov / codecov/project succeeded Nov 15, 2024 in 0s

73.29% (+0.09%) compared to 22a89e8

View this Pull Request on Codecov

73.29% (+0.09%) compared to 22a89e8

Details

Codecov Report

Attention: Patch coverage is 66.45570% with 53 lines in your changes missing coverage. Please review.

Project coverage is 73.29%. Comparing base (22a89e8) to head (bc0acdd).

Files with missing lines Patch % Lines
enterprise/reporting/reporting.go 16.66% 27 Missing and 3 partials ⚠️
enterprise/reporting/event_sampler.go 75.26% 19 Missing and 4 partials ⚠️
Additional details and impacted files
@@                    Coverage Diff                     @@
##           feat.reports-bactching    #5298      +/-   ##
==========================================================
+ Coverage                   73.19%   73.29%   +0.09%     
==========================================================
  Files                         424      426       +2     
  Lines                       59949    60107     +158     
==========================================================
+ Hits                        43879    44054     +175     
+ Misses                      13606    13591      -15     
+ Partials                     2464     2462       -2     

☔ View full report in Codecov by Sentry.
📢 Have feedback on the report? Share it here.