Fix no-duplicate-selectors false negatives for three or more duplicates

[erosman]

What minimal example or steps are needed to reproduce the bug?

a[href*="abc"],
a[href*=""],
a[href*=""],
a[href*=""] {
  color: #fd0;
}

What minimal configuration is needed to reproduce the bug?

default

How did you run Stylelint?

Demo

Which Stylelint-related dependencies are you using?

{
  "stylelint": "16.3.0",
  "stylelint-config-standard": "36.0.0"
}

What did you expect to happen?

Error should occur on line 3 & 4 and refer to line 2 e.g.
Line 2 selector is not a duplicate of line 1 selector.

---

PS. (on another note) Is there a rule for empty selector value? 🤔

What actually happened?

ERROR Unexpected duplicate selector "a[href*=""]", first used at line 1 (no-duplicate-selectors) [2:1-12]

Do you have a proposal to fix the bug?

No response

[ybiquitous]

@erosman Thanks for the report. As you commented, this is a bug.

I've labeled the issue as ready to implement. Please consider contributing if you have time.

There are steps on how to fix a bug in a rule in the Developer guide.

[ybiquitous]

PS. (on another note) Is there a rule for empty selector value? 🤔

Sadly, we have no such a rule in the built-in rules. If you're interested, feel free to open a new issue.

[ybiquitous]

For your reference, this bug is not only for empty values.

a[href*="abc"], /* this line only is reported */
a[href*="abc"],
a[href*="abc"],
a[href*=""] {
  color: #fd0;
}

See the demo.

[erosman]

@erosman Thanks for the report. As you commented, this is a bug.

I've labeled the issue as ready to implement. Please consider contributing if you have time.

There are steps on how to fix a bug in a rule in the Developer guide.

Is the code for the rule in /lib/rules/no-duplicate-selectors/index.mjs?

TBH, my writing style is considerably different for me to contribute to the code.

[ybiquitous]

Is the code for the rule in /lib/rules/no-duplicate-selectors/index.mjs?

Yes.

TBH, my writing style is considerably different for me to contribute to the code.

Regarding coding style, we keep consistency thanks to Prettier and ESLint.

[erosman]

BTW, is CLA required by StyleLint?
(It prevented me from processing a PR in ESLint.)

[ybiquitous]

No, CLA is not required.

[github-actions]

This issue is older than one month. Please ask before opening a pull request, as it may no longer be relevant.