Allow to disable retriable blocks

[fatkodima]

This is primarily for testing convenience, but can be very useful in development, for example, when you want to quickly disable/reenable retrying.

[hound]

Style/RedundantBegin: Redundant begin block detected.

[fatkodima]

This is needed because ruby < 2.5 requires explicit begin...ensure inside blocks.

[kamui]

You could just do:

# config/initializers/retriable.rb
Retriable.configure do |c|
  if Rails.env.development?
    c.tries = 1
  end
end

Right?

[fatkodima]

Yes, for simple cases.

I honestly do not remember, which specific example I had in mind. But from README changes you can see, how it also simplifies working with contexts. Sure, you can write something like this for each context:

Retriable.configure do |c|
   c.contexts[:google_api] = {
       tries:         Rails.env.test? ? 1 : 5,
       base_interval: 3,
       on:            [
           Net::ReadTimeout,
           Signet::AuthorizationError,
           Errno::ECONNRESET,
           OpenSSL::SSL::SSLError
       ]
   }
 end

But the proposed solution looks cleaner for me.