search

SonarSource / SonarTS

Static code analyzer for TypeScript
GNU Lesser General Public License v3.0
761 stars 53 forks source link

typescript:S4798 in abstract classes / functions #890

Closed Franziskus1988 closed 4 years ago

Franziskus1988 commented 4 years ago

I want to report a bug.

SonarTS version: 5.14 (build 18788)
Node.js version: v10.14.1
TypeScript version: 3.5.2

SonarQube version: 7.9.1.27448
SonarLint version and used IDE: 1.12.0 VSCode

Rule key: typescript:S4798

Error: Sonarlint shows this warning:

sonar_warning

But you can't set the value in a abstract class:

tsc_error

Expected behavior Think it's a false positive.

vilchik-elena commented 4 years ago

Thanks for reporting! I created an issue in SonarJS project https://github.com/SonarSource/SonarJS/issues/1768 (now TS analysis is performed by SonarJS)