Silence some ruby warnings

[ojab]

Description

Describe your changes:

…/sentry-ruby-core-4.6.1/lib/sentry/core_ext/object/duplicable.rb:122: warning: method redefined; discarding old duplicable?
…/activesupport-6.1.4/lib/active_support/core_ext/object/duplicable.rb:36: warning: previous definition of duplicable? was here
…/sentry-ruby-core-4.6.1/lib/sentry/core_ext/object/deep_dup.rb:31: warning: method redefined; discarding old deep_dup
…/activesupport-6.1.4/lib/active_support/core_ext/object/deep_dup.rb:29: warning: previous definition of deep_dup was here
…/sentry-ruby-core-4.6.1/lib/sentry/transport/configuration.rb:14: warning: method redefined; discarding old transport_class=
…/sentry-ruby-core-4.6.1/lib/sentry/event.rb:78: warning: method redefined; discarding old timestamp=
…/sentry-ruby-core-4.6.1/lib/sentry/event.rb:82: warning: method redefined; discarding old level=
…/sentry-ruby-core-4.6.1/lib/sentry/event.rb:102: warning: method redefined; discarding old type
…/sentry-ruby-core-4.6.1/lib/sentry/transaction_event.rb:17: warning: method redefined; discarding old start_timestamp=
…/sentry-ruby-core-4.6.1/lib/sentry/transaction_event.rb:21: warning: method redefined; discarding old type

Not sure if we should undef_method or just not define our variants. As a corporation you probably want undef_method for extra-defined behavior.
Also could add warning gem to devdeps and enforce no-warnings via CI if desired.

[ojab]

Sending to review without green CI due to First-time contributors need a maintainer to approve running workflows.. Four specs are failing locally both on master and here.

Also not sure if changelog here is needed and which CHANGELOG.md should it go to.

[codecov-commenter]

Codecov Report

Merging #1504 (0adc8d0) into master (dbd2cc8) will increase coverage by 0.54%.
The diff coverage is 100.00%.

@@            Coverage Diff             @@
##           master    #1504      +/-   ##
==========================================
+ Coverage   98.19%   98.74%   +0.54%     
==========================================
  Files         218      123      -95     
  Lines       10598     6760    -3838     
==========================================
- Hits        10407     6675    -3732     
+ Misses        191       85     -106     

Impacted Files	Coverage Δ
sentry-ruby/lib/sentry/configuration.rb	97.96% <100.00%> (ø)
sentry-ruby/lib/sentry/core_ext/object/deep_dup.rb	94.11% <100.00%> (+0.36%)	⬆️
...ntry-ruby/lib/sentry/core_ext/object/duplicable.rb	70.21% <100.00%> (+0.64%)	⬆️
sentry-ruby/lib/sentry/transport/configuration.rb	100.00% <100.00%> (ø)
sentry-ruby/spec/sentry/configuration_spec.rb	98.24% <100.00%> (ø)
...y/spec/sentry/interfaces/request_interface_spec.rb	99.14% <100.00%> (ø)
sentry-ruby/spec/sentry/net/http_spec.rb	100.00% <100.00%> (ø)
sentry-ruby/spec/sentry/transport_spec.rb	100.00% <100.00%> (ø)
...raven/spec/raven/processors/utf8conversion_spec.rb
sentry-raven/lib/raven/processor/cookies.rb
... and 93 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 dbd2cc8...0adc8d0. Read the comment docs.

[st0012]

Thanks for the PR 👍
The changes generally look good but I want to cleanup Event and TransactionEvent's attributes myself. Can you exclude them?

And regarding changeslog, can you update the top-level one? thanks.

[ojab]

The changes generally look good but I want to cleanup Event and TransactionEvent's attributes myself. Can you exclude them?

Done. Any ETA on this or GH issue that I could follow?

[st0012]

👍

[st0012]

Done. Any ETA on this or GH issue that I could follow?

I'll add a PR for that in a week and include an changelog entry for this PR as well.

[ojab]

Ugh, I have CHANGELOG entry locally, but it's not committed 🤦

[st0012]

I've added #1513 as a continuation of the work.