Skip to content

Notify users when their custom options are discarded#2303

Merged
st0012 merged 2 commits intomasterfrom
followup-#2301
Apr 29, 2024
Merged

Notify users when their custom options are discarded#2303
st0012 merged 2 commits intomasterfrom
followup-#2301

Conversation

@st0012
Copy link
Copy Markdown
Contributor

@st0012 st0012 commented Apr 29, 2024

This is a followup on #2301

It should save users some debugging if we can tell them what options they pass are discarded and how they can fix it.

The warning is only enabled at debug level so it should not be too noisy.

It should save users from some debugging if we can tell them what options
they pass are discarded and how they can fix it.

The warning is only enabled at debug level so it should not be too noisy.
@st0012 st0012 requested a review from sl0thentr0py April 29, 2024 11:38
Copy link
Copy Markdown
Member

@sl0thentr0py sl0thentr0py left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

thx!

@codecov
Copy link
Copy Markdown

codecov bot commented Apr 29, 2024

Codecov Report

All modified and coverable lines are covered by tests ✅

Project coverage is 98.71%. Comparing base (07aaf5e) to head (e32620d).

Additional details and impacted files
@@           Coverage Diff           @@
##           master    #2303   +/-   ##
=======================================
  Coverage   98.71%   98.71%           
=======================================
  Files         202      202           
  Lines       13294    13301    +7     
=======================================
+ Hits        13123    13130    +7     
  Misses        171      171           
Components Coverage Δ
sentry-ruby 99.09% <100.00%> (+<0.01%) ⬆️
sentry-rails 97.37% <ø> (ø)
sentry-sidekiq 96.98% <ø> (ø)
sentry-resque 96.76% <ø> (ø)
sentry-delayed_job 98.91% <ø> (ø)
sentry-opentelemetry 100.00% <ø> (ø)
Files Coverage Δ
sentry-ruby/lib/sentry/hub.rb 99.38% <100.00%> (+<0.01%) ⬆️
sentry-ruby/lib/sentry/scope.rb 100.00% <100.00%> (ø)
sentry-ruby/spec/sentry/hub_spec.rb 96.86% <100.00%> (+0.03%) ⬆️
sentry-ruby/spec/sentry/scope_spec.rb 100.00% <100.00%> (ø)

@st0012 st0012 merged commit 6cdb1fc into master Apr 29, 2024
@st0012 st0012 deleted the followup-#2301 branch April 29, 2024 11:44
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants