From 0833c52aa08cbb86322eeb6f02c1ab76c1d72a6c Mon Sep 17 00:00:00 2001 From: =?UTF-8?q?=C3=8Dtalo=20Matos?= Date: Fri, 12 Dec 2025 01:35:38 -0300 Subject: [PATCH] Refactor: use idiomatic .last instead of .order(:desc).first (#2098) Simplifies the last_event method in ActivitySpike::Detector by using the more idiomatic Rails pattern .order(:created_at).last instead of .order(created_at: :desc).first. Both generate the same SQL query but .last is more readable and conventional in Rails codebases. --- app/models/card/activity_spike/detector.rb | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/app/models/card/activity_spike/detector.rb b/app/models/card/activity_spike/detector.rb index b533318e3..1ba44f501 100644 --- a/app/models/card/activity_spike/detector.rb +++ b/app/models/card/activity_spike/detector.rb @@ -51,6 +51,6 @@ class Card::ActivitySpike::Detector end def last_event - card.events.order(created_at: :desc).first + card.events.order(:created_at).last end end