Eric Morand
Eric Morand
The RSPEC documentation of rule S6850 use the term "heading" instead of the term "header". This PR aligns the plugin documentation of the rule accordingly.
The file `packages/jsts/src/rules/S6606/unit.test.ts` of our SonarJS project triggers the S2187 rule even though it actually contains test cases. See: https://next.sonarqube.com/sonarqube/project/issues?issues=5746b5c9-cbdb-4dcc-99ac-be04e97b65f1&open=5746b5c9-cbdb-4dcc-99ac-be04e97b65f1&id=org.sonarsource.javascript%3Ajavascript
Add an exception for React components: * ignore when returning null or false **This issue was extracted from #4527 to not delay the sanitation fix.**
### Reproducer: ```ts async function foo () { await bar(); // FN } function bar () { return true } ``` ### Analysis: TypeScript considers `false` as a type union...
## 10.12.0 Improve top dismissed JS/TS rules, and add one rule (S2004): https://github.com/SonarSource/SonarJS/releases
## Review A dedicated reviewer checked the rule description successfully for: - [ ] logical errors and incorrect information - [ ] information gaps and missing content - [ ]...
Methods `fromMapFileComment` and `fromMapFileSource` take a `readMap` function as second argument, that is expected to return either a `string` or a Promise of a `string`. But it is not clear...
Testing the pattern that consists at importing the actual Babel modules instead of referencing their packages by their name, as supported by Babel: https://babeljs.io/docs/options#plugin-and-preset-options https://babeljs.io/docs/options#entrytarget It allows dependency resolvers to...
This fix improves the documentation based on the answer to this question: https://github.com/eslint/eslint/discussions/18965 #### Prerequisites checklist - [X] I have read the [contributing guidelines](https://github.com/eslint/eslint/blob/HEAD/CONTRIBUTING.md). #### What is the purpose of...