diff options
author | Koichi ITO <koic.ito@gmail.com> | 2020-01-18 00:42:12 +0900 |
---|---|---|
committer | Koichi ITO <koic.ito@gmail.com> | 2020-01-18 00:58:17 +0900 |
commit | 0dde4b4e3055ca918651f974b055e2cf7447fdba (patch) | |
tree | dc7b559682834249173a3b7d62fd2b5c5b120098 | |
parent | 7fa93b19a098f90019fbe336ffcae4c3dbe4f54f (diff) | |
download | hashie-0dde4b4e3055ca918651f974b055e2cf7447fdba.tar.gz |
Suppress a Ruby's warning when using Ruby 2.6.0+
This PR suppresses the following warning that `deep_merge` method and
`deep_update` method are defined twice when using Ruby 2.6.0+.
```console
% bundle exec rake
(snip)
/Users/koic/src/github.com/hahie/hashie/lib/hashie/mash.rb:226: warning:
method redefined; discarding old deep_merge
/Users/koic/src/github.com/hahie/hashie/lib/hashie/mash.rb:212: warning:
previous definition of deep_merge was here
/Users/koic/src/github.com/hahie/hashie/lib/hashie/mash.rb:232: warning:
method redefined; discarding old deep_update
/Users/koic/src/github.com/hahie/hashie/lib/hashie/mash.rb:218: warning:
previous definition of deep_update was here
```
-rw-r--r-- | CHANGELOG.md | 1 | ||||
-rw-r--r-- | lib/hashie/extensions/ruby_version_check.rb | 6 | ||||
-rw-r--r-- | lib/hashie/mash.rb | 28 |
3 files changed, 20 insertions, 15 deletions
diff --git a/CHANGELOG.md b/CHANGELOG.md index d6ddaad..7f71fae 100644 --- a/CHANGELOG.md +++ b/CHANGELOG.md @@ -38,6 +38,7 @@ scheme are considered to be bugs. * [#510](https://github.com/hashie/hashie/pull/510): Ensure that `Hashie::Mash#compact` is only defined on Ruby version >= 2.4.0 - [@bobbymcwho](https://github.com/bobbymcwho). * [#511](https://github.com/hashie/hashie/pull/511): Suppress keyword arguments warning for Ruby 2.7.0 - [@koic](https://github.com/koic). * [#512](https://github.com/hashie/hashie/pull/512): Suppress an integer unification warning for using Ruby 2.4.0+ - [@koic](https://github.com/koic). +* [#513](https://github.com/hashie/hashie/pull/513): Suppress a Ruby's warning when using Ruby 2.6.0+ - [@koic](https://github.com/koic). * Your contribution here. ### Security diff --git a/lib/hashie/extensions/ruby_version_check.rb b/lib/hashie/extensions/ruby_version_check.rb index bcd3436..b287230 100644 --- a/lib/hashie/extensions/ruby_version_check.rb +++ b/lib/hashie/extensions/ruby_version_check.rb @@ -9,7 +9,11 @@ module Hashie module ClassMethods def with_minimum_ruby(version) - yield if RubyVersion.new(RUBY_VERSION) >= RubyVersion.new(version) + yield if with_minimum_ruby?(version) + end + + def with_minimum_ruby?(version) + RubyVersion.new(RUBY_VERSION) >= RubyVersion.new(version) end end end diff --git a/lib/hashie/mash.rb b/lib/hashie/mash.rb index 194a6f8..96b24c9 100644 --- a/lib/hashie/mash.rb +++ b/lib/hashie/mash.rb @@ -207,20 +207,7 @@ module Hashie alias include? key? alias member? key? - # Performs a deep_update on a duplicate of the - # current mash. - def deep_merge(other_hash, &blk) - dup.deep_update(other_hash, &blk) - end - - # Recursively merges this mash with the passed - # in hash, merging each hash in the hierarchy. - def deep_update(other_hash, &blk) - _deep_update(other_hash, &blk) - self - end - - with_minimum_ruby('2.6.0') do + if with_minimum_ruby?('2.6.0') # Performs a deep_update on a duplicate of the # current mash. def deep_merge(*other_hashes, &blk) @@ -235,6 +222,19 @@ module Hashie end self end + else + # Performs a deep_update on a duplicate of the + # current mash. + def deep_merge(other_hash, &blk) + dup.deep_update(other_hash, &blk) + end + + # Recursively merges this mash with the passed + # in hash, merging each hash in the hierarchy. + def deep_update(other_hash, &blk) + _deep_update(other_hash, &blk) + self + end end # Alias these lexically so they get the correctly defined |