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

Only instantiate SessionFlusher when the SDK is enabled under the current env #2245

Merged
merged 2 commits into from
Feb 10, 2024

Update changelog

1aaaa60
Select commit
Loading
Failed to load commit list.
Merged

Only instantiate SessionFlusher when the SDK is enabled under the current env #2245

Update changelog
1aaaa60
Select commit
Loading
Failed to load commit list.
Codecov / codecov/project succeeded Feb 10, 2024 in 0s

97.43% (+30.97%) compared to 2979e86

View this Pull Request on Codecov

97.43% (+30.97%) compared to 2979e86

Details

Codecov Report

Merging #2245 (1aaaa60) into master (2979e86) will increase coverage by 30.97%.
The diff coverage is 100.00%.

Additional details and impacted files
@@             Coverage Diff             @@
##           master    #2245       +/-   ##
===========================================
+ Coverage   66.46%   97.43%   +30.97%     
===========================================
  Files         101      102        +1     
  Lines        3793     3825       +32     
===========================================
+ Hits         2521     3727     +1206     
+ Misses       1272       98     -1174     
Components Coverage Δ
sentry-ruby 98.14% <100.00%> (+42.53%) ⬆️
sentry-rails 95.05% <ø> (ø)
sentry-sidekiq 94.70% <ø> (ø)
sentry-resque 92.06% <ø> (-1.59%) ⬇️
sentry-delayed_job 95.60% <ø> (ø)
sentry-opentelemetry 100.00% <ø> (ø)
Files Coverage Δ
sentry-ruby/lib/sentry-ruby.rb 96.44% <100.00%> (+32.48%) ⬆️
sentry-ruby/lib/sentry/configuration.rb 98.79% <100.00%> (+19.52%) ⬆️

... and 53 files with indirect coverage changes