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

Enable opting out of patches #2151

Merged
merged 1 commit into from Oct 30, 2023
Merged

Enable opting out of patches #2151

merged 1 commit into from Oct 30, 2023

Conversation

sl0thentr0py
Copy link
Member

@sl0thentr0py sl0thentr0py commented Oct 25, 2023

closes #2095

@codecov
Copy link

codecov bot commented Oct 25, 2023

Codecov Report

Merging #2151 (efc4691) into master (2e51449) will increase coverage by 0.05%.
The diff coverage is 92.85%.

❗ Current head efc4691 differs from pull request most recent head 3f46bef. Consider uploading reports for the commit 3f46bef to get more accurate results

@@            Coverage Diff             @@
##           master    #2151      +/-   ##
==========================================
+ Coverage   97.27%   97.33%   +0.05%     
==========================================
  Files          97       97              
  Lines        3634     3638       +4     
==========================================
+ Hits         3535     3541       +6     
+ Misses         99       97       -2     
Components Coverage Δ
sentry-ruby 98.02% <92.85%> (+0.04%) ⬆️
sentry-rails 94.98% <ø> (ø)
sentry-sidekiq 93.70% <ø> (ø)
sentry-resque 93.65% <ø> (+1.58%) ⬆️
sentry-delayed_job 94.36% <ø> (ø)
sentry-opentelemetry 100.00% <ø> (ø)
Files Coverage Δ
sentry-ruby/lib/sentry-ruby.rb 96.37% <100.00%> (+0.51%) ⬆️
sentry-ruby/lib/sentry/configuration.rb 98.73% <100.00%> (+0.01%) ⬆️
sentry-ruby/lib/sentry/net/http.rb 100.00% <100.00%> (ø)
sentry-ruby/lib/sentry/puma.rb 92.30% <100.00%> (ø)
sentry-ruby/lib/sentry/redis.rb 96.22% <66.66%> (+0.07%) ⬆️

... and 2 files with indirect coverage changes

@sl0thentr0py sl0thentr0py force-pushed the neel/patches-opt-out branch 2 times, most recently from afcdaf6 to fa97407 Compare October 27, 2023 15:02
@sl0thentr0py sl0thentr0py marked this pull request as ready for review October 27, 2023 15:02
Copy link
Collaborator

@st0012 st0012 left a comment

Choose a reason for hiding this comment

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

I think it works well 👍 Just suggestions on tests.

sentry-ruby/spec/sentry_spec.rb Outdated Show resolved Hide resolved
sentry-ruby/spec/sentry_spec.rb Outdated Show resolved Hide resolved
Copy link
Collaborator

@st0012 st0012 left a comment

Choose a reason for hiding this comment

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

👍

@sl0thentr0py sl0thentr0py merged commit 1a74734 into master Oct 30, 2023
96 checks passed
@sl0thentr0py sl0thentr0py deleted the neel/patches-opt-out branch October 30, 2023 18:16
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.

Allow opting out of patches
2 participants