AbstractFormatter and AbstractRule are now abstract classes

refs palantir#631
zdychacek · Sep 16, 2015 · 18e9e89 · 18e9e89
1 parent b93535b
commit 18e9e89
Show file tree

Hide file tree

Showing 2 changed files with 4 additions and 10 deletions.
diff --git a/src/language/formatter/abstractFormatter.ts b/src/language/formatter/abstractFormatter.ts
@@ -15,9 +15,7 @@
  */
 
 module Lint.Formatters {
-    export class AbstractFormatter implements Lint.IFormatter {
-        public format(failures: Lint.RuleFailure[]): string {
-            throw Lint.abstract();
-        }
+    export abstract class AbstractFormatter implements Lint.IFormatter {
+        public abstract format(failures: Lint.RuleFailure[]): string;
     }
 }
diff --git a/src/language/rule/abstractRule.ts b/src/language/rule/abstractRule.ts
@@ -15,7 +15,7 @@
  */
 
 module Lint.Rules {
-    export class AbstractRule implements Lint.IRule {
+    export abstract class AbstractRule implements Lint.IRule {
         private value: any;
         private options: Lint.IOptions;
 
@@ -38,11 +38,7 @@ module Lint.Rules {
             return this.options;
         }
 
-        /* tslint:disable:no-unused-variable */
-        public apply(sourceFile: ts.SourceFile): RuleFailure[] {
-            throw Lint.abstract();
-        }
-        /* tslint:enable:no-unused-variable */
+        public abstract apply(sourceFile: ts.SourceFile): RuleFailure[];
 
         public applyWithWalker(walker: Lint.RuleWalker): RuleFailure[] {
             walker.walk(walker.getSourceFile());