Yassin Kammoun

Add S2933 (`prefer-readonly`) to Sonar way

3

S2933 is in Sonar way for C# but not for TypeScript.

This helper definition should be removed https://github.com/SonarSource/SonarJS/blob/538ae84783d08e8dc73bdfef0aaa402d069ce914/packages/jsts/src/rules/helpers/ast.ts#L416 and calls to it should now invoke https://github.com/SonarSource/SonarJS/blob/538ae84783d08e8dc73bdfef0aaa402d069ce914/packages/jsts/src/rules/helpers/ast.ts#L448 _Ticket created from a TODO leftover in the codebase._

Do not generate UCFGs for JavaScript code inside HTML

2

We are wrongly generating UCFGs for JavaScript code embedded inside HTML files. This is due to the fact that the analysis of JavaScript code inside HTML is reusing the same...

The comment-based testing framework was originally a partial port to TypeScript from [sonar-analyzer-commons's Java implementation](https://github.com/SonarSource/sonar-analyzer-commons/tree/master/test-commons/src/main/java/org/sonarsource/analyzer/commons/checks/verifier). Due to lack of time, the porting effort was stopped midway. We currently are able...

JS-362 Enable SonarJaRED to consume SonarJS ASTs without SonarArmor

1

I suggest we move forward with this change for now. There is no reason for JaRED to depend on Armor. However, let's raise the topic of the relevance of these...