@Rub21 commented on this pull request.


> +  def put_object_request
+    @service.client.client.api_requests.find { |request| 
request[:operation_name] == :put_object }
+  end

Good point  to renamed it, but  rfind is only in Ruby 4.0, so it may fails on 
Ruby 3.3. We  could use `reverse_each.find` instead.

```rb
def find_latest_put_object_request
  @service.client.client.api_requests.reverse_each.find { |request| 
request[:operation_name] == :put_object }
end
```



-- 
Reply to this email directly or view it on GitHub:
https://github.com/openstreetmap/openstreetmap-website/pull/7124#discussion_r3454048638
You are receiving this because you are subscribed to this thread.

Message ID: 
<openstreetmap/openstreetmap-website/pull/7124/review/[email protected]>
_______________________________________________
rails-dev mailing list
[email protected]
https://lists.openstreetmap.org/listinfo/rails-dev

Reply via email to