Remove Future#==

Author: byroot

CHANGELOG.md

@@ -7,6 +7,7 @@
 - Removed `reconnect_delay_max` and `reconnect_delay`, you can pass precise sleep durations to `reconnect_attempts` instead.
 - Require Ruby 2.5+.
 - Removed the deprecated `queue` and `commit` methods. Use `pipelined` instead.
+- Removed the deprecated `Redis::Future#==`.
 - Removed the deprecated `pipelined` and `multi` signature. Commands now MUST be called on the block argument, not the original redis instance.
 - Removed `Redis.current`. You shouldn't assume there is a single global Redis connection, use a connection pool instead,
   and libaries using Redis should accept a Redis instance (or connection pool) as a config. E.g. `MyLibrary.redis = Redis.new(...)`.

lib/redis/pipeline.rb

@@ -85,16 +85,6 @@ def initialize(command, coerce)
       @coerce = coerce
     end
 
-    def ==(_other)
-      message = +"The methods == and != are deprecated for Redis::Future and will be removed in 5.0.0"
-      message << " - You probably meant to call .value == or .value !="
-      message << " (#{::Kernel.caller(1, 1).first})\n"
-
-      ::Redis.deprecate!(message)
-
-      super
-    end
-
     def inspect
       "<Redis::Future #{@command.inspect}>"
     end

test/redis/pipelining_commands_test.rb

@@ -135,15 +135,6 @@ def test_futures_raise_when_command_errors_and_needs_transformation
     end
   end
 
-  def test_futures_warn_when_tested_for_equality
-    r.pipelined do |p|
-      @result = p.sadd("foo", 1)
-    end
-
-    Redis.expects(:deprecate!).once
-    @result == 1
-  end
-
   def test_futures_can_be_identified
     r.pipelined do |p|
       @result = p.sadd("foo", 1)