Fix deprecated uses of `Redis#pipelined` #516

casperisfine · 2022-01-25T10:00:11Z

The following is deprecated

redis.pipelined do
  redis.get(key)
end

And should be rewritten as:

redis.pipelined do |pipeline|
  pipeline.get(key)
end

Functionally it makes no difference.

Context: redis/redis-rb#1059 The following is deprecated ```ruby redis.pipelined do redis.get(key) end ``` And should be rewritten as: ```ruby redis.pipelined do |pipeline| pipeline.get(key) end ``` Functionally it makes no difference.

OsamaSayegh · 2022-02-10T19:55:45Z

Thank you @casperisfine!

Fix deprecated uses of Redis#pipelined

e8b2e80

Context: redis/redis-rb#1059 The following is deprecated ```ruby redis.pipelined do redis.get(key) end ``` And should be rewritten as: ```ruby redis.pipelined do |pipeline| pipeline.get(key) end ``` Functionally it makes no difference.

OsamaSayegh approved these changes Feb 10, 2022

View reviewed changes

OsamaSayegh merged commit 275c932 into MiniProfiler:master Feb 10, 2022

Provide feedback

Saved searches

Use saved searches to filter your results more quickly

Fix deprecated uses of `Redis#pipelined` #516

Fix deprecated uses of `Redis#pipelined` #516

casperisfine commented Jan 25, 2022

OsamaSayegh commented Feb 10, 2022

Fix deprecated uses of Redis#pipelined #516

Fix deprecated uses of Redis#pipelined #516

Conversation

casperisfine commented Jan 25, 2022

OsamaSayegh commented Feb 10, 2022

Fix deprecated uses of `Redis#pipelined` #516

Fix deprecated uses of `Redis#pipelined` #516