ActiveSupport::TimeWithZone preference for rails users in pagy_calendar_minmax?

[benkoshy]

Throwing up some ideas:

If we are using Rails, pagy_calendar_minmax might typically be implemented as follows:

def pagy_calendar_minmax(collection)
    ordered_collection = collection.order(:created_at)
    min_time = ordered_collection.first.created_at
    max_time = ordered_collection.last.created_at
    return [min_time.to_time, max_time.to_time]
end

Would the following be more complicated?

def pagy_calendar_minmax(collection)
    # min_time is an ActiveSupport::TimeWithZone convenience for rails users.

    ordered_collection = collection.order(:created_at)
    min_time = ordered_collection.first.created_at
    max_time = ordered_collection.last.created_at

    return [min_time, max_time] ## would be handy
    return [min_time.to_time, max_time.to_time] ## or if we wanted to use time, then we can do so as well
  end

Why?

• ActiveSupport::TimeWithZone might be preferred by rails users.

[ddnexus]

There are less verbose suggestions in the doc of 5.5.
They look pretty straightforward for rails users.