added legacy linter warning logging

CHANGELOG.md

@@ -8,6 +8,7 @@ All notable changes to the "prettier-vscode" extension will be documented in thi
 
 - Removed AppInsights telemetry as its no longer needed
 - Updated instructions on how to use linters
+- Added log warning about legacy linter integrations
 
 ## [4.2.0]
 

README.md

@@ -290,3 +290,7 @@ When a `package.json` is present in your project and it contains prettier, plugi
 **Your project is configured to use an outdated version of prettier that cannot be used by this extension. Upgrade to the latest version of prettier.**
 
 You must upgrade to a newer version of prettier.
+
+**You are using a legacy linter integration that will be removed in the future.**
+
+Support for prettier-eslint, prettier-tslint, and prettier-stylelint will be removed in a future version of this extension. You should switch to one of the [supported and recommended ways of integrating Prettier with linters](https://github.com/prettier/prettier-vscode#linter-integration).

src/PrettierEditService.ts

@@ -14,7 +14,7 @@ import { ConfigResolver, RangeFormattingOptions } from "./ConfigResolver";
 import { IgnorerResolver } from "./IgnorerResolver";
 import { LanguageResolver } from "./LanguageResolver";
 import { LoggingService } from "./LoggingService";
-import { INVALID_PRETTIER_CONFIG } from "./message";
+import { INVALID_PRETTIER_CONFIG, LEGACY_LINTER_INTEGRATION } from "./message";
 import { ModuleResolver } from "./ModuleResolver";
 import { NotificationService } from "./NotificationService";
 import { PrettierEditProvider } from "./PrettierEditProvider";
@@ -326,6 +326,7 @@ export default class PrettierEditService implements Disposable {
 
       if (prettierTslintModule) {
         this.loggingService.logInfo("Formatting using 'prettier-tslint'");
+        this.loggingService.logWarning(LEGACY_LINTER_INTEGRATION);
         return this.safeExecution(() => {
           const prettierTslintFormat = prettierTslintModule.format as PrettierTslintFormat;
 
@@ -345,6 +346,7 @@ export default class PrettierEditService implements Disposable {
       );
       if (prettierEslintModule) {
         this.loggingService.logInfo("Formatting using 'prettier-eslint'");
+        this.loggingService.logWarning(LEGACY_LINTER_INTEGRATION);
         return this.safeExecution(() => {
           const prettierEslintFormat = prettierEslintModule as PrettierEslintFormat;
 
@@ -364,6 +366,7 @@ export default class PrettierEditService implements Disposable {
       );
       if (prettierStylelintModule) {
         this.loggingService.logInfo("Formatting using 'prettier-stylelint'");
+        this.loggingService.logWarning(LEGACY_LINTER_INTEGRATION);
         const prettierStylelint = prettierStylelintModule as IPrettierStylelint;
         return this.safeExecution(
           prettierStylelint.format({

src/message.ts

@@ -5,3 +5,5 @@ export const FAILED_TO_LOAD_MODULE_MESSAGE =
   "Failed to load module. If you have prettier or plugins referenced in package.json, ensure you have run `npm install`";
 export const INVALID_PRETTIER_CONFIG =
   "Invalid prettier configuration file detected. See log for details.";
+export const LEGACY_LINTER_INTEGRATION =
+  "You are using a legacy linter integration that will be removed in the future. See the documentation on setting up linters: https://github.com/prettier/prettier-vscode#linter-integration";