Dont mix positional and named args

be686e12 · Sean Carroll · e4bfca48 · be686e12
Commit be686e12 authored Aug 30, 2019 by Sean Carroll
Hide whitespace changes
Inline Side-by-side

Showing with 1 addition and 1 deletion

lib/gitlab/optimistic_locking.rb lib/gitlab/optimistic_locking.rb +1 -1

No files found.
--- a/lib/gitlab/optimistic_locking.rb
+++ b/lib/gitlab/optimistic_locking.rb
@@ -4,7 +4,7 @@ module Gitlab
  module OptimisticLocking
    module_function

-    def retry_lock(subject, retries: nil, &block)
+    def retry_lock(subject, retries = nil, &block)
      retries ||= 100
      # TODO(Observability): We should be recording details of the number of retries and the duration of the total execution here
      ActiveRecord::Base.transaction do