Remove rotate from the list of call_names

In some cases it returns `self` and not a copy.
This commit is contained in:
Sijawusz Pur Rahnama 2021-01-22 12:20:50 +01:00
parent 5f6c14c9d6
commit a219a73258

View file

@ -33,7 +33,6 @@ module Ameba::Rule::Performance
# - sort_by
# - shuffle
# - reverse
# - rotate
# ```
class ChainedCallsWithNoBang < Base
properties do
@ -42,7 +41,7 @@ module Ameba::Rule::Performance
# All of those have bang method variants returning `self`
# and are not modifying the receiver type (like `compact` does),
# thus are safe to switch to the bang variant.
call_names : Array(String) = %w(uniq sort sort_by shuffle reverse rotate)
call_names : Array(String) = %w(uniq sort sort_by shuffle reverse)
end
# All these methods are allocating a new object