Tibor Blenessy comments

Results 39 comments of


                                            Tibor Blenessy

S3776 should exclude the complexity of JSX attribute short-circuits and nested function definitions

we didn't manage to do this in current sprint, removing from milestone

Do not mention custom rules when deploying security frontend

Is this coming from internal feedback?

Importing ESlint reports shouldn't spam output with errors if no report found

sorry, I mixed the issues, this comment was meant for #1985

Create rule: Assertions comparing incompatible types should not be made

I created draft PR #2848 , which is unfinished. We currently don't support type resolution in the new test framework. I decided to postpone this rule for later.

Avoid deployment if NodeJS is not available

30 seconds are rather abnormal, this happens on Windows due to antivirus settings. I think on linux machines it is much much faster, otherwise we should check how we could...

Improve S2189 (no-infinite-loop) by reporting only on local vars

There are multiple FP patterns for the `no-unmodified-loop` rule we are using, I think we should consider rewriting the rule completely. Some of the FP patterns are documented below ```...

Analyse TS files without tsconfig

We've decided not to implement this, because having tsconfig file is requirement for any TS project, so the use case of this change is limited only to some toy examples...

Support wildcard patterns for sonar.typescript.tsconfigPath

Related to https://services.sonarsource.com/browse/SUPPORT-16890 (internal link)

ERROR: Failed to find a source file matching path <path> in program created with <path_to_tsconfig>

hello @sivantha96 , can you please post your problem on our community forum https://community.sonarsource.com/ ? This is legacy repository and analysis is not done using this plugin anymore. It's likely...

no-useless-cast false positive

hello @eranshabi , I am not able to reproduce this issue with TypeScript 3.4.5 . This rule just uses information from TypeScript compiler, and consequently, it will depend on the...