New: Add rule no-any

Author: dannyfritz

README.md

@@ -51,3 +51,4 @@ Then configure the rules you want to use under the rules section.
 
 * `typescript/type-annotation-spacing` - enforces one space after the colon and zero spaces before the colon of a type annotation.
 * `typescript/explicit-member-accessibility` - enforces accessibility modifiers on class properties and methods.
+* `typescript/no-any` - enforces the any type is not used.

docs/rules/no-any.md

@@ -0,0 +1,40 @@
+# Enforces the any type is not used (no-any)
+
+Using the any type defeats the purpose of using TypeScript.
+When any is used, all compiler type checks around that value are ignored.
+
+## Rule Details
+
+This rule aims to keep TypeScript maximally useful.
+
+The following patterns are considered warnings:
+
+```ts
+const age : any = "seventeen"
+```
+
+```ts
+function greet () : any {}
+```
+
+The following patterns are not warnings:
+
+```ts
+const age : number = 17
+```
+
+```ts
+function greet () : string {}
+```
+
+## When Not To Use It
+
+If you require an unknown type or if you pull in a library without typings.
+
+## Further Reading
+
+* TypeScript [any type](https://www.typescriptlang.org/docs/handbook/basic-types.html#any)
+
+## Compatibility
+
+* TSLint: [no-any](https://palantir.github.io/tslint/rules/no-any/)

lib/rules/no-any.js

@@ -0,0 +1,69 @@
+/**
+ * @fileoverview Enforces the any type is not used.
+ * @author Danny Fritz
+ */
+"use strict";
+
+//------------------------------------------------------------------------------
+// Rule Definition
+//------------------------------------------------------------------------------
+
+module.exports = {
+    meta: {
+        docs: {
+            description: "Enforces the any type is not used.",
+            category: "TypeScript"
+        },
+        schema: []
+    },
+
+    create: function(context) {
+
+        //----------------------------------------------------------------------
+        // Helpers
+        //----------------------------------------------------------------------
+
+        /**
+         * Checks if the any type is used
+         * @param {ASTNode} typeAnnotation The TypeAnnotation node.
+         * @returns {void}
+         * @private
+         */
+        function checkTypeAnnotationForAny(typeAnnotation) {
+            if (typeAnnotation.typeAnnotation.type === "TSAnyKeyword") {
+                context.report({
+                    node: typeAnnotation,
+                    message: "Unexpected any. Specify a different type."
+                });
+            }
+        }
+
+        /**
+         * Checks if a function node used the any type
+         * @param {ASTNode} node The node representing a function.
+         * @returns {void}
+         * @private
+         */
+        function checkFunctionReturnTypeForAny(node) {
+            if (node.returnType) {
+                checkTypeAnnotationForAny(node.returnType);
+            }
+        }
+
+        //----------------------------------------------------------------------
+        // Public
+        //----------------------------------------------------------------------
+        return {
+
+            Identifier(node) {
+                if (node.typeAnnotation) {
+                    checkTypeAnnotationForAny(node.typeAnnotation);
+                }
+            },
+
+            FunctionDeclaration: checkFunctionReturnTypeForAny,
+            FunctionExpression: checkFunctionReturnTypeForAny,
+            ArrowFunctionExpression: checkFunctionReturnTypeForAny
+        };
+    }
+};

tests/lib/rules/no-any.js

@@ -0,0 +1,52 @@
+/**
+ * @fileoverview Enforces the any type is not used
+ * @author Danny Fritz
+ */
+"use strict";
+
+//------------------------------------------------------------------------------
+// Requirements
+//------------------------------------------------------------------------------
+
+var rule = require("../../../lib/rules/no-any"),
+    RuleTester = require("eslint").RuleTester;
+
+
+//------------------------------------------------------------------------------
+// Tests
+//------------------------------------------------------------------------------
+
+var ruleTester = new RuleTester();
+ruleTester.run("no-any", rule, {
+
+    valid: [
+        {
+            code: "const number : number = 1",
+            parser: "typescript-eslint-parser"
+        },
+        {
+            code: "function greet () : string {}",
+            parser: "typescript-eslint-parser"
+        }
+    ],
+    invalid: [
+        {
+            code: "const number : any = 1",
+            parser: "typescript-eslint-parser",
+            errors: [{
+                message: "Unexpected any. Specify a different type.",
+                line: 1,
+                column: 16
+            }]
+        },
+        {
+            code: "function generic () : any {}",
+            parser: "typescript-eslint-parser",
+            errors: [{
+                message: "Unexpected any. Specify a different type.",
+                line: 1,
+                column: 23
+            }]
+        }
+    ]
+});