Eric Morand

Here is an additional false positive that is most probably triggered by this issue: https://community.sonarsource.com/t/rule-javascript-s2189-false-positive-in-loop-exit-condition/109778

For information, we are working on it. There is no ETA yet, but as soon as we have one, we'll update this issue, so stay tuned.

I was not able to reproduce the first issue (_React components, ignore when returning null or false_): based on my testings with the current state of SonarJS, the S3800 rule...

Verified locally against SonarQube 10.4, using the latest plugin artifact.

@ilia-kebets-sonarsource , if this is of some help, there is a very typical usage of types redundancy in the typings of DOM events, and the purpose is to give developers...

Can we re-discuss this issue just to make sure that it is legit to add an exception? * With types, if `strictNullChecks` is disabled, the rule never triggers because `T`...

@snigdha920 , thanks for the report. I assume you are extending `plugin:sonarjs/recommended`. If so, can you please try extending `plugin:sonarjs/recommended-legacy` instead? This is documented [there](https://github.com/SonarSource/eslint-plugin-sonarjs?tab=readme-ov-file#usage).

Actually, the documentation was partially wrong, which may explain why you missed it. We are fixing it right now.

Hi @ocsoares, You can use the SonarJS plugin exactly like you would any other plugin, following the instructions in the [ESLint plugin documentation](https://eslint.org/docs/latest/use/configure/plugins). For example, if you want to use...