sindresorhus/eslint-plugin-unicorn
Rule proposal: `no-set-duplicates`
Closed
#1,559 opened on Oct 25, 2021
help wantednew rule
Repository metrics
- Stars
- (5,022 stars)
- PR merge metrics
- (Avg merge 1d 16h) (399 merged PRs in 30d)
Description
Sometimes a hardcoded Set grows large enough for it to be hard to see if there is already a value in it. One could miss that a value one wants to add is already there and add a duplicate.
This rule should at minimum report hardcoded sets of literals and consts with duplicates.
This rule may also report consecutive .add calls with same arguments (this is probably much harder to implement). Or maybe this could be a separate rule like unicorn/no-array-push-push.
Fail
const allowed = new Set([
'foo',
'bar',
'buz',
'qux',
'bar', // ← Remove duplicate value `'bar'` from the Set
]);
Pass
const allowed = new Set([
'foo',
'bar',
'buz',
'qux',
]);