Don't check declared vars in the unused-variable-rule

fix palantir#190
DickvdBrink · May 13, 2015 · 8d73fae · 8d73fae
1 parent b08678f
commit 8d73fae
Show file tree

Hide file tree

Showing 2 changed files with 4 additions and 1 deletion.
diff --git a/src/rules/noUnusedVariableRule.ts b/src/rules/noUnusedVariableRule.ts
@@ -95,7 +95,8 @@ class NoUnusedVariablesWalker extends Lint.RuleWalker {
 
     // skip exported variables
     public visitVariableStatement(node: ts.VariableStatement): void {
-        if (this.hasModifier(node.modifiers, ts.SyntaxKind.ExportKeyword)) {
+        if (this.hasModifier(node.modifiers, ts.SyntaxKind.ExportKeyword) ||
+            this.hasModifier(node.modifiers, ts.SyntaxKind.DeclareKeyword)) {
             this.skipVariableDeclaration = true;
         }
 

diff --git a/test/files/rules/nounusedvariable-var.test.ts b/test/files/rules/nounusedvariable-var.test.ts
@@ -16,3 +16,5 @@ try {
 } catch (e) {
   // e is unused but that's still ok
 }
+
+declare var tmp: any;