You signed in with another tab or window. Reload to refresh your session.You signed out in another tab or window. Reload to refresh your session.You switched accounts on another tab or window. Reload to refresh your session.Dismiss alert
{{ message }}
This repository has been archived by the owner on Apr 27, 2021. It is now read-only.
Hello @antonovicha,
thanks for the feedback and the clear description.
In SonarTS 2.0, we are going to change this rule, and base it on no-duplicate-imports, from ESLint.
The bad news is that this FP is also present in ESlint. Fixing it on our side does not make much sense, they could definitely benefits from this.
Therefore I advise you to report it to ESLint instead.
import * as express from "express";
import { Request, Response} from "express"; // False-positive here
@antonovicha
Why don't we access like express.Request , express.Response instead of importing 2nd line.
Is it something related with tree shaking or compiler(babel/webpack etc)/typescript version (before 2.7/after 2.7 etc) related case?
I want to report a bug.
SonarTS version: 1.9 (build 3766)
Node.js version: 8.11
TypeScript version: 3.5.1
Rule key: S3863
Reproducer
Expected behavior
No error is reported. It's not possible to merge namespace and named imports. See similar issue in tslint: palantir/tslint#3418
The text was updated successfully, but these errors were encountered: