palantir · nchen63 · Mar 28, 2017 · Mar 28, 2017
diff --git a/src/configs/latest.ts b/src/configs/latest.ts
@@ -19,7 +19,6 @@
 // tslint:disable:object-literal-key-quotes
 export const rules = {
     // added in v3.x
-    "no-invalid-this": false,
     "no-angle-bracket-type-assertion": true,
 
     // added in v4.1

diff --git a/src/configs/recommended.ts b/src/configs/recommended.ts
@@ -81,7 +81,6 @@ export const rules = {
     "no-reference": true,
     "no-shadowed-variable": true,
     "no-string-literal": true,
-    "no-switch-case-fall-through": false,
     "no-trailing-whitespace": true,
     "no-unsafe-finally": true,
     "no-unused-expression": true,
@@ -126,7 +125,6 @@ export const rules = {
     "semicolon": {
         options: ["always"],
     },
-    "switch-default": true,
     "trailing-comma": {
         options: {
             multiline: "always",
@@ -213,7 +211,6 @@ export const jsRules = {
     "no-reference": true,
     "no-shadowed-variable": true,
     "no-string-literal": true,
-    "no-switch-case-fall-through": false,
     "no-trailing-whitespace": true,
     "no-unused-expression": true,
     // disable this rule as it is very heavy performance-wise and not that useful
@@ -241,7 +238,6 @@ export const jsRules = {
     "semicolon": {
         options: ["always"],
     },
-    "switch-default": true,
     "trailing-comma": {
         options: {
             multiline: "always",

diff --git a/src/rules/noInferrableTypesRule.ts b/src/rules/noInferrableTypesRule.ts
@@ -89,7 +89,8 @@ class NoInferrableTypesWalker extends Lint.AbstractWalker<IOptions> {
                         hasModifier(node.modifiers, ts.SyntaxKind.ReadonlyKeyword)) {
                         break;
                     }
-                    /* falls through*/
+                    this.checkDeclaration(node as ts.VariableLikeDeclaration);
+                    break;
                 case ts.SyntaxKind.VariableDeclaration:
                     this.checkDeclaration(node as ts.VariableLikeDeclaration);
             }

diff --git a/src/rules/noInvalidThisRule.ts b/src/rules/noInvalidThisRule.ts
diff --git a/src/rules/noSwitchCaseFallThroughRule.ts b/src/rules/noSwitchCaseFallThroughRule.ts
diff --git a/src/rules/noUnnecessaryQualifierRule.ts b/src/rules/noUnnecessaryQualifierRule.ts
@@ -72,7 +72,8 @@ class Walker extends Lint.ProgramAwareRuleWalker {
                     this.visitNamespaceAccess(node, expression, name);
                     break;
                 }
-                // falls through
+                super.visitNode(node);
+                break;
             default:
                 super.visitNode(node);
         }

diff --git a/src/rules/trailingCommaRule.ts b/src/rules/trailingCommaRule.ts
@@ -98,7 +98,8 @@ class TrailingCommaWalker extends Lint.AbstractWalker<Options> {
                     if ((node as ts.NewExpression).arguments === undefined) {
                         break;
                     }
-                    // falls through
+                    this.checkList((node as ts.CallExpression | ts.NewExpression).arguments!, node.end);
+                    break;
                 case ts.SyntaxKind.CallExpression:
                     this.checkList((node as ts.CallExpression | ts.NewExpression).arguments!, node.end);
                     break;

diff --git a/src/tsconfig.json b/src/tsconfig.json
@@ -1,6 +1,7 @@
 {
     "compilerOptions": {
         "module": "commonjs",
+        "noFallthroughCasesInSwitch": true,
         "noImplicitAny": true,
         "noImplicitReturns": true,
         "noImplicitThis": true,

diff --git a/test/rules/no-invalid-this/enabled/test.js.lint b/test/rules/no-invalid-this/enabled/test.js.lint