Currently, we only report cases that Array only called .includes(). But in some other cases, it should still be preferred.
const foo = [1, 2, 3];
if (foo.includes(1));
// Still work the same if we use `Set`
for (const element of foo);
const bar = [...foo, somethingElse];
call(...foo);
const length = foo.length; // Use `.size` when switch to Set
// Even this, if the callback don't use the second parameter, they are the same
foo.forEach(element => call(element));We only need make sure the original Array have unique values.
sindresorhus / eslint-plugin-unicorn
More than 100 powerful ESLint rules
4.2k
How does funding with Polar work?
1
Pay now to fund the work behind this issue.
2
Get updates on progress being made.
3
Maintainer is rewarded once the issue is completed.
FAQ
Backer
You're funding impactful open source efforts
Contributor
You want to contribute to this effort
Maintainer
You want to get funding like this too