`prefer-spread`: Add prefer spread over `.slice()/.concat()`

Currently, prefer-spread only check Array.from(), how about add prefer spread over .slice()/.concat()?

Fail

const foo = array.slice();

const foo = array.concat(item);

const foo = array.concat(anotherArray);

Pass

const foo = [...array];

const foo = array.slice(1);

The concat() case, we can’t know item is array or not.

Maybe with suggestions? Suggestion 1: [...array, itemOrArray] Suggestion 2: [...array, ...itemOrArray]

Another problem is TypedArray#slice() and String#concat(). String#slice should not a problem. Nobody use string.slice()(no arguments).

---

Any other method people use to clone an array?

[sindresorhus]

@coreyfarrell You can just use [value].flat() for that (we will eventually enforce that https://github.com/sindresorhus/eslint-plugin-unicorn/issues/975).

[sindresorhus]

@fisker Maybe we can make it opt-in? I have personally never used or seen .slice() used on TypedArray. For me, it’s more valuable to catch Array#slice(). // eslint-disable are good for edge-cases like this.