Skip checking declared functions for no-unused-variable (fixes #434)

- also update tsconfig project files

src/rules/noUnusedVariableRule.ts

@@ -135,11 +135,11 @@ class NoUnusedVariablesWalker extends Lint.RuleWalker {
         this.skipParameterDeclaration = false;
     }
 
-    // check function declarations (skipping exports)
+    // skip exported and declared functions
     public visitFunctionDeclaration(node: ts.FunctionDeclaration): void {
         var variableName = node.name.text;
 
-        if (!Lint.hasModifier(node.modifiers, ts.SyntaxKind.ExportKeyword)) {
+        if (!Lint.hasModifier(node.modifiers, ts.SyntaxKind.ExportKeyword, ts.SyntaxKind.DeclareKeyword)) {
             this.validateReferencesForVariable(variableName, node.name.getStart());
         }
 

src/rules/tsconfig.json

@@ -57,6 +57,7 @@
         "./quotemarkRule.ts",
         "./radixRule.ts",
         "./semicolonRule.ts",
+        "./sortObjectLiteralKeysRule.ts",
         "./switchDefaultRule.ts",
         "./tripleEqualsRule.ts",
         "./typedefRule.ts",

test/files/rules/nounusedvariable-function.test.ts

@@ -13,3 +13,5 @@ function func3() {
 export function func4() {
     return func1(2, 3);
 }
+
+declare function func5(): any;

test/tsconfig.json

@@ -64,11 +64,13 @@
         "./rules/noUnusedExpressionRuleTests.ts",
         "./rules/noUnusedVariableRuleTests.ts",
         "./rules/noUseBeforeDeclareRuleTests.ts",
+        "./rules/noVarKeywordRuleTests.ts",
         "./rules/noVarRequiresRuleTests.ts",
         "./rules/oneLineRuleTests.ts",
         "./rules/quotemarkRuleTests.ts",
         "./rules/radixRuleTests.ts",
         "./rules/semicolonRuleTests.ts",
+        "./rules/sortObjectLiteralKeysRuleTests.ts",
         "./rules/switchDefaultRuleTests.ts",
         "./rules/tripleEqualsRuleTests.ts",
         "./rules/typedefRuleTests.ts",