remove block indentation checks

- along with 8eab92b, fixes palantir#16

Author: ashwinr

README.md

@@ -15,7 +15,7 @@ Supported Rules
 * `eqeqeq` enforces === and !== in favor of == and !=.
 * `evil` disallows `eval` function invocations.
 * `forin` enforces a `for ... in` statement to be filtered with an `if` statement.*
-* `indent` enforces consistent indentation levels for the whole file.
+* `indent` enforces consistent indentation levels.
 * `labelpos` enforces labels only on sensible statements.
 * `label-undefined` checks that labels are defined before usage.
 * `maxlen` sets the maximum length of a line.

bin/tslint-cli.js

@@ -27096,12 +27096,6 @@ var Lint;
                 this.currentLevel = 0;
                 this.tabWidth = tabWidth;
             }
-            IndentWalker.prototype.visitBlock = function (node) {
-                this.visitToken(node.openBraceToken);
-                this.checkAndVisitList(node.statements);
-                this.visitToken(node.closeBraceToken);
-            };
-
             IndentWalker.prototype.visitClassDeclaration = function (node) {
                 this.currentLevel += 1;
                 _super.prototype.visitClassDeclaration.call(this, node);

lib/tslint.js

@@ -27096,12 +27096,6 @@ var Lint;
                 this.currentLevel = 0;
                 this.tabWidth = tabWidth;
             }
-            IndentWalker.prototype.visitBlock = function (node) {
-                this.visitToken(node.openBraceToken);
-                this.checkAndVisitList(node.statements);
-                this.visitToken(node.closeBraceToken);
-            };
-
             IndentWalker.prototype.visitClassDeclaration = function (node) {
                 this.currentLevel += 1;
                 _super.prototype.visitClassDeclaration.call(this, node);

package.json

@@ -1,6 +1,6 @@
 {
   "name": "tslint",
-  "version": "0.2.7",
+  "version": "0.2.8",
   "description": "a static analysis linter for TypeScript",
   "bin": {
     "tslint": "./bin/tslint"

src/rules/indentRule.ts

@@ -49,13 +49,6 @@ module Lint.Rules {
             this.tabWidth = tabWidth;
         }
 
-        // block indentation
-        public visitBlock(node: TypeScript.BlockSyntax): void {
-            this.visitToken(node.openBraceToken);
-            this.checkAndVisitList(node.statements);
-            this.visitToken(node.closeBraceToken);
-        }
-
         // class indentation
         public visitClassDeclaration(node: TypeScript.ClassDeclarationSyntax): void {
             this.currentLevel += 1;

test/rules/indentRuleTests.ts

@@ -33,11 +33,6 @@ describe("<indent>", () => {
             Lint.Test.assertContainsFailure(actualFailures, expectedFailure);
         });
 
-        it("enforces function indentation", () => {
-            var expectedFailure = Lint.Test.createFailure(fileName, [62, 4], [62, 19], failureString + "9");
-            Lint.Test.assertContainsFailure(actualFailures, expectedFailure);
-        });
-
         it("enforces class variable indentation", () => {
             var expectedFailure = Lint.Test.createFailure(fileName, [66, 3], [66, 10], failureString + "2");
             Lint.Test.assertContainsFailure(actualFailures, expectedFailure);
@@ -67,18 +62,6 @@ describe("<indent>", () => {
             Lint.Test.assertContainsFailure(actualFailures, expectedFailure3);
             Lint.Test.assertContainsFailure(actualFailures, expectedFailure4);
         });
-
-        it("enforces block indentation", () => {
-            var expectedFailure1 = Lint.Test.createFailure(fileName, [94, 3], [94, 9], failureString + "2");
-            var expectedFailure2 = Lint.Test.createFailure(fileName, [98, 4], [98, 10], failureString + "6");
-            var expectedFailure3 = Lint.Test.createFailure(fileName, [102, 3], [102, 9], failureString + "5");
-            var expectedFailure4 = Lint.Test.createFailure(fileName, [106, 5], [106, 11], failureString + "7");
-
-            Lint.Test.assertContainsFailure(actualFailures, expectedFailure1);
-            Lint.Test.assertContainsFailure(actualFailures, expectedFailure2);
-            Lint.Test.assertContainsFailure(actualFailures, expectedFailure3);
-            Lint.Test.assertContainsFailure(actualFailures, expectedFailure4);
-        });
     });
 
     describe("on a space-indented file", () => {
@@ -94,11 +77,6 @@ describe("<indent>", () => {
             Lint.Test.assertContainsFailure(actualFailures, expectedFailure);
         });
 
-        it("enforces function indentation", () => {
-            var expectedFailure = Lint.Test.createFailure(fileName, [62, 10], [62, 25], failureString + "9");
-            Lint.Test.assertContainsFailure(actualFailures, expectedFailure);
-        });
-
         it("enforces class variable indentation", () => {
             var expectedFailure = Lint.Test.createFailure(fileName, [66, 3], [66, 10], failureString + "2");
             Lint.Test.assertContainsFailure(actualFailures, expectedFailure);
@@ -128,17 +106,5 @@ describe("<indent>", () => {
             Lint.Test.assertContainsFailure(actualFailures, expectedFailure3);
             Lint.Test.assertContainsFailure(actualFailures, expectedFailure4);
         });
-
-        it("enforces block indentation", () => {
-            var expectedFailure1 = Lint.Test.createFailure(fileName, [94, 3], [94, 9], failureString + "2");
-            var expectedFailure2 = Lint.Test.createFailure(fileName, [98, 7], [98, 13], failureString + "6");
-            var expectedFailure3 = Lint.Test.createFailure(fileName, [102, 6], [102, 12], failureString + "5");
-            var expectedFailure4 = Lint.Test.createFailure(fileName, [106, 8], [106, 14], failureString + "7");
-
-            Lint.Test.assertContainsFailure(actualFailures, expectedFailure1);
-            Lint.Test.assertContainsFailure(actualFailures, expectedFailure2);
-            Lint.Test.assertContainsFailure(actualFailures, expectedFailure3);
-            Lint.Test.assertContainsFailure(actualFailures, expectedFailure4);
-        });
     });
 });