remove confusing deep_merge

Author: grosser

CHANGELOG

@@ -1,3 +1,5 @@
+ * removed deep_merge and deeper_merge, if you used them replace with `!` variant with `preserve_unmergeables: true`
+
 2022-01-07 Jason Frey <[email protected]>
   * Ship version 1.2.2
 

README.md

@@ -174,7 +174,6 @@ In a Gemfile:
 
 The deep_merge methods will then be defined as
 
-    Hash#deeper_merge
     Hash#deeper_merge!
     Hash#ko_deeper_merge!
 

lib/deep_merge/deep_merge_hash.rb

@@ -13,13 +13,6 @@ def deep_merge!(source, options = {})
       default_opts = {:preserve_unmergeables => false}
       DeepMerge::deep_merge!(source, self, default_opts.merge(options))
     end
-
-    # deep_merge will merge and skip any unmergeables in destination hash
-    def deep_merge(source, options = {})
-      default_opts = {:preserve_unmergeables => true}
-      DeepMerge::deep_merge!(source, self, default_opts.merge(options))
-    end
-
   end # DeepMergeHashExt
 end
 

lib/deep_merge/rails_compat.rb

@@ -13,12 +13,6 @@ def deeper_merge!(source, options = {})
       default_opts = {:preserve_unmergeables => false}
       DeepMerge::deep_merge!(source, self, default_opts.merge(options))
     end
-
-    # deeper_merge will merge and skip any unmergeables in destination hash
-    def deeper_merge(source, options = {})
-      default_opts = {:preserve_unmergeables => true}
-      DeepMerge::deep_merge!(source, self, default_opts.merge(options))
-    end
   end
 end
 

test/test_deep_merge.rb

@@ -35,7 +35,7 @@ def test_hash_deep_merge
 
     hash_src  = {'id' => 'xxx'}
     hash_dest = {'id' => [1,2,3]}
-    assert hash_dest.deep_merge(hash_src)
+    assert hash_dest.deep_merge!(hash_src, preserve_unmergeables: true)
     assert_equal({'id' => [1,2,3]}, hash_dest)
   end