📎 noUnusedVariables should report unused imports

[Conaclos]

Description

As discussed in #4188, the current implementation of noUnusedVariables does not report unused imports as in the following example:

import { unused } from "module"
import defaultUnused, * as unusedNamespace from "module"

The equivalent ESLint rule no-unused-var reports unused imports.
I didn't see any reason why noUnusedVariables ignores unused imports.

noUnusedVariables should report unused imports.

[arendjr]

For TypeScript users there's a work-around: you can also use noUnusedLocals in your tsconfig.json and it will also report them.

Still agree Rome should also be able to handle this. Might try to give this a go if I have some time.

[arendjr]

It seems the current rule in main actually does report unused imports now. There's an invalidImport.ts test case even already. Just to make double sure, I added the snippet to the test cases:

import { unused } from "module";
import defaultUnused, * as unusedNamespace from "module";

But all 3 were reported by the rule too, so I think this is safe to close.