You signed in with another tab or window. Reload to refresh your session.You signed out in another tab or window. Reload to refresh your session.You switched accounts on another tab or window. Reload to refresh your session.Dismiss alert
Define these instance methods of Business::Calendar:
earliest_business_date_on_or_after – alias for roll_forward
latest_business_date_on_or_before – alias for roll_backward
While these names are more verbose, my team agrees that they are easier to understand.
Part of the problem with the current names roll_forward and roll_backward is that the imperative verb “roll” makes the methods sound like they will mutate the passed date, though they do not. (It’s good that they do not.)
Define these new names as aliases instead of renaming the existing methods to preserve backwards compatibility.
The text was updated successfully, but these errors were encountered:
Define these instance methods of
Business::Calendar
:earliest_business_date_on_or_after
– alias forroll_forward
latest_business_date_on_or_before
– alias forroll_backward
While these names are more verbose, my team agrees that they are easier to understand.
Part of the problem with the current names
roll_forward
androll_backward
is that the imperative verb “roll” makes the methods sound like they will mutate the passed date, though they do not. (It’s good that they do not.)Define these new names as aliases instead of renaming the existing methods to preserve backwards compatibility.
The text was updated successfully, but these errors were encountered: