Replace TSLint comments with ESLint comments

Author: andyleejordan

src/controls/checkboxQuickPick.ts

@@ -75,7 +75,7 @@ function showInner(
             if (index >= 0) {
                 toggleSelection(items[index]);
             } else {
-                // tslint:disable-next-line:no-console
+                // eslint-disable-next-line no-console
                 console.log(`Couldn't find CheckboxQuickPickItem for label '${selection.label}'`);
             }
 

src/features/CustomViews.ts

@@ -74,7 +74,7 @@ class PowerShellContentProvider implements vscode.TextDocumentContentProvider {
     private viewIndex: { [id: string]: CustomView } = {};
     private didChangeEvent: vscode.EventEmitter<vscode.Uri> = new vscode.EventEmitter<vscode.Uri>();
 
-    // tslint:disable-next-line:member-ordering
+    // eslint-disable-next-line @typescript-eslint/member-ordering
     public onDidChange: vscode.Event<vscode.Uri> = this.didChangeEvent.event;
 
     public provideTextDocumentContent(uri: vscode.Uri): string {

src/features/ExtensionCommands.ts

@@ -93,7 +93,7 @@ export const GetEditorContextRequestType =
     new RequestType<IGetEditorContextRequestArguments, IEditorContext, void>(
         "editor/getEditorContext");
 
-// tslint:disable-next-line:no-empty-interface
+// eslint-disable-next-line @typescript-eslint/no-empty-interface
 export interface IGetEditorContextRequestArguments {
 }
 

src/logging.ts

@@ -187,7 +187,7 @@ export class Logger implements ILogger {
                 timestampedMessage + os.EOL,
                 (err) => {
                     if (err) {
-                        // tslint:disable-next-line:no-console
+                        // eslint-disable-next-line no-console
                         console.log(`Error writing to vscode-powershell log file: ${err}`);
                     }
                 });

src/main.ts

@@ -35,7 +35,7 @@ import { PowerShellLanguageId } from "./utils";
 import { LanguageClientConsumer } from "./languageClientConsumer";
 
 // The most reliable way to get the name and version of the current extension.
-// tslint:disable-next-line: no-var-requires
+// eslint-disable-next-line @typescript-eslint/no-var-requires
 const PackageJSON: any = require("../../package.json");
 
 // the application insights key (also known as instrumentation key) used for telemetry.

src/session.ts

@@ -776,7 +776,7 @@ class SessionMenuItem implements vscode.QuickPickItem {
 
     constructor(
         public readonly label: string,
-        // tslint:disable-next-line:no-empty
+        // eslint-disable-next-line no-empty, @typescript-eslint/no-empty-function
         public readonly callback: () => void = () => {}) {
     }
 }

src/settings.ts

@@ -264,7 +264,7 @@ export function load(): ISettings {
         notebooks:
             configuration.get<INotebooksSettings>("notebooks", defaultNotebooksSettings),
         startAsLoginShell:
-            // tslint:disable-next-line
+            // eslint-disable-next-line
             // We follow the same convention as VS Code - https://github.com/microsoft/vscode/blob/ff00badd955d6cfcb8eab5f25f3edc86b762f49f/src/vs/workbench/contrib/terminal/browser/terminal.contribution.ts#L105-L107
             //   "Unlike on Linux, ~/.profile is not sourced when logging into a macOS session. This
             //   is the reason terminals on macOS typically run login shells by default which set up

test/features/ExternalApi.test.ts

@@ -6,7 +6,7 @@ import * as vscode from "vscode";
 import { before, beforeEach, afterEach } from "mocha";
 import { IExternalPowerShellDetails, IPowerShellExtensionClient } from "../../src/features/ExternalApi";
 
-// tslint:disable-next-line: no-var-requires
+// eslint-disable-next-line @typescript-eslint/no-var-requires
 const PackageJSON: any = require("../../../package.json");
 const testExtensionId = `${PackageJSON.publisher}.${PackageJSON.name}`;
 

test/runTests.ts

@@ -5,7 +5,7 @@ import * as path from "path";
 
 import { runTests } from "vscode-test";
 
-// tslint:disable-next-line: no-var-requires
+// eslint-disable-next-line @typescript-eslint/no-var-requires
 const PackageJSON: any = require("../../package.json");
 const testExtensionId = `${PackageJSON.publisher}.${PackageJSON.name}`;
 
@@ -31,9 +31,9 @@ async function main() {
             version: "insiders"
         });
     } catch (err) {
-        // tslint:disable-next-line:no-console
+        // eslint-disable-next-line no-console
         console.error(err);
-        // tslint:disable-next-line:no-console
+        // eslint-disable-next-line no-console
         console.error("Failed to run tests");
         process.exit(1);
     }