noUnusedVariableRule should apply the ignorePattern to imports

src/rules/noUnusedVariableRule.ts

@@ -42,7 +42,7 @@ export class Rule extends Lint.Rules.TypedRule {
                 * NOTE: this option is experimental and does not work with classes
                 that use abstract method declarations, among other things.
             * \`{"ignore-pattern": "pattern"}\` where pattern is a case-sensitive regexp.
-            Variable names that match the pattern will be ignored.`,
+            Variable names and imports that match the pattern will be ignored.`,
         options: {
             type: "array",
             items: {
@@ -114,6 +114,13 @@ function walk(ctx: Lint.WalkContext<Options>, program: ts.Program): void {
 
         const failure = ts.flattenDiagnosticMessageText(diag.messageText, "\n");
 
+        if (ignorePattern !== undefined) {
+            const varName = /'(.*)'/.exec(failure)![1];
+            if (ignorePattern.test(varName)) {
+                continue;
+            }
+        }
+
         if (kind === UnusedKind.VARIABLE_OR_PARAMETER) {
             const importName = findImport(diag.start, sourceFile);
             if (importName !== undefined) {
@@ -129,13 +136,6 @@ function walk(ctx: Lint.WalkContext<Options>, program: ts.Program): void {
             }
         }
 
-        if (ignorePattern !== undefined) {
-            const varName = /'(.*)'/.exec(failure)![1];
-            if (ignorePattern.test(varName)) {
-                continue;
-            }
-        }
-
         ctx.addFailureAt(diag.start, diag.length!, failure);
     }
 

test/rules/no-unused-variable/ignore-pattern/a.test.ts

@@ -0,0 +1 @@
+export class _A {}

test/rules/no-unused-variable/ignore-pattern/var.ts.lint

@@ -1,3 +1,5 @@
+import { _A } from './a.test';
+
 var x = 3;
 
 var y = x;