sindresorhus/eslint-plugin-unicorn

Rule proposal: `prefer-change-array-by-copy`

Closed

#1,514 opened on Sep 8, 2021

 (14 comments) (14 reactions) (0 assignees)JavaScript (468 forks)user submission
help wantednew rule

Repository metrics

Stars
 (5,022 stars)
PR merge metrics
 (Avg merge 1d 16h) (399 merged PRs in 30d)

Description

"Change Array by copy" proposal

Fail

array.reverse();
array.sort();
const array = [...foo];
array.splice(0, 1);
const array = [...foo];
array[1] = 'changed';

Pass

array = array.toReversed();
array = array.toSorted();
const array = foo.toSpliced(0, 1);
const array = foo.with(1, 'changed');

Contributor guide