Deprecated our --fix formatters

Starts on #577
microsoft · Oct 24, 2018 · c328bce · c328bce
1 parent e076605
commit c328bce
Show file tree

Hide file tree

Showing 2 changed files with 15 additions and 1 deletion.
diff --git a/src/fixNoRequireImportsFormatter.ts b/src/fixNoRequireImportsFormatter.ts
@@ -3,6 +3,8 @@
 import {RuleFailure} from 'tslint';
 import {BaseFormatter} from './utils/BaseFormatter';
 
+let warnedForDeprecation = false;
+
 /**
  * Formatter that fixes your unused imports.
  */
@@ -11,6 +13,11 @@ export class Formatter extends BaseFormatter {
 /* tslint:enable:export-name */
 
     constructor() {
+        if (!warnedForDeprecation) {
+            console.warn('The fix-no-var-keyword formatter is deprecated. Use --fix instead.');
+            warnedForDeprecation = true;
+        }
+
         super('no-require-imports', function (this: Formatter, failure: RuleFailure): void {
             const fileName: string = failure.getFileName();
             const fileContents: string = this.readFile(fileName);

diff --git a/src/fixNoVarKeywordFormatter.ts b/src/fixNoVarKeywordFormatter.ts
@@ -3,14 +3,21 @@
 import {RuleFailure} from 'tslint';
 import {BaseFormatter} from './utils/BaseFormatter';
 
+let warnedForDeprecation = false;
+
 /**
- * Formatter that fixes your unused imports.
+ * Formatter that fixes your var keywords.
  */
 /* tslint:disable:export-name */
 export class Formatter extends BaseFormatter {
 /* tslint:enable:export-name */
 
     constructor() {
+        if (!warnedForDeprecation) {
+            console.warn('The fix-no-var-keyword formatter is deprecated. Use --fix instead.');
+            warnedForDeprecation = true;
+        }
+
         super('no-var-keyword', function (this: Formatter, failure: RuleFailure): void {
             const fileName: string = failure.getFileName();
             const fileContents: string = this.readFile(fileName);