Prevent error of sonarjs/no-duplicate-string with CSS classes (Tailwind, Foundation, Boostrap...) about eslint-plugin-sonarjs HOT 9 OPEN

Thanks a lot - that indeed makes sense, we will look at it.

from eslint-plugin-sonarjs.

@yassin-kammoun-sonarsource now I on vacation and I can't provide you an real example but when I will go back to my work I will show you real code that have this problem

from eslint-plugin-sonarjs.

To bump, it would be sweet to be able to pass a regex to ignoreStrings

from eslint-plugin-sonarjs.

Would this option meet your needs?

from eslint-plugin-sonarjs.

Hi!, thanks but I think that doesn't meet my need because Tailwind has hundreds or thousands of classes 🙈

from eslint-plugin-sonarjs.

@alexander-kamushkin-sonarsource great thanks! It would be awesome if the implementation can be made for css classes in general because in this case others CSS frameworks will work

from eslint-plugin-sonarjs.

@RodrigoTomeES I analyzed the snippet you provided and the rule doesn't report any issue on it. I also checked the rule implementation and it already makes an exception to all JSX attributes.

Was this a feature suggestion or are you facing an actual false positive from the rule?

from eslint-plugin-sonarjs.

I have the same issue. Mostly happens when for me with tailwind-merge using an array of classes like this:

And yes, it might be an abomination to pass classes as an array, but it lets prettier format them nicely.

from eslint-plugin-sonarjs.