Move ESLint next-line directives

Author: andyleejordan

eslint.config.mjs

@@ -29,6 +29,7 @@ export default tseslint.config(
                 "error",
                 {
                     argsIgnorePattern: "^_",
+                    destructuredArrayIgnorePattern: "^_",
                 },
             ],
             "@typescript-eslint/no-unsafe-argument": "off",

src/extension.ts

@@ -66,8 +66,8 @@ export async function activate(
         PowerShellLanguageId,
         {
             // TODO: Remove the useless escapes
-            // eslint-disable-next-line no-useless-escape
             wordPattern:
+                // eslint-disable-next-line no-useless-escape
                 /(-?\d*\.\d\w*)|([^\`\~\!\@\#\%\^\&\*\(\)\=\+\[\{\]\}\\\|\;\'\"\,\.\<\>\/\?\s]+)/g,
 
             indentationRules: {

src/features/ExpandAlias.ts

@@ -69,9 +69,9 @@ export class ExpandAliasFeature extends LanguageClientConsumer {
         );
     }
 
-    // eslint-disable-next-line @typescript-eslint/no-empty-function
     public override onLanguageClientSet(
         _languageClient: LanguageClient,
+        // eslint-disable-next-line @typescript-eslint/no-empty-function
     ): void {}
 
     public dispose(): void {

src/features/ExternalApi.ts

@@ -65,7 +65,6 @@ export class ExternalApiFeature implements IPowerShellExtensionClient {
             `Registering extension '${id}' for use with API version '${apiVersion}'.`,
         );
 
-        // eslint-disable-next-line @typescript-eslint/no-unused-vars
         for (const [
             _name,
             externalExtension,

src/features/HelpCompletion.ts

@@ -101,9 +101,9 @@ class TriggerFinder {
     public updateState(document: TextDocument, changeText: string): void {
         switch (this.state) {
             case SearchState.Searching:
-                // eslint-disable-next-line @typescript-eslint/prefer-string-starts-ends-with
                 if (
                     changeText.length === 1 &&
+                    // eslint-disable-next-line @typescript-eslint/prefer-string-starts-ends-with
                     changeText[0] === this.triggerCharacters[this.count]
                 ) {
                     this.state = SearchState.Locked;
@@ -113,10 +113,10 @@ class TriggerFinder {
                 break;
 
             case SearchState.Locked:
-                // eslint-disable-next-line @typescript-eslint/prefer-string-starts-ends-with
                 if (
                     document === this.document &&
                     changeText.length === 1 &&
+                    // eslint-disable-next-line @typescript-eslint/prefer-string-starts-ends-with
                     changeText[0] === this.triggerCharacters[this.count]
                 ) {
                     this.count++;
@@ -156,9 +156,9 @@ class HelpCompletionProvider extends LanguageClientConsumer {
         return this.triggerFinderHelpComment.found;
     }
 
-    // eslint-disable-next-line @typescript-eslint/no-empty-function
     public override onLanguageClientSet(
         _languageClient: LanguageClient,
+        // eslint-disable-next-line @typescript-eslint/no-empty-function
     ): void {}
 
     public updateState(

src/features/RemoteFiles.ts

@@ -54,9 +54,9 @@ export class RemoteFilesFeature extends LanguageClientConsumer {
         });
     }
 
-    // eslint-disable-next-line @typescript-eslint/no-empty-function
     public override onLanguageClientSet(
         _languageClient: LanguageClient,
+        // eslint-disable-next-line @typescript-eslint/no-empty-function
     ): void {}
 
     public dispose(): void {

src/features/ShowHelp.ts

@@ -46,9 +46,9 @@ export class ShowHelpFeature extends LanguageClientConsumer {
         );
     }
 
-    // eslint-disable-next-line @typescript-eslint/no-empty-function
     public override onLanguageClientSet(
         _languageClient: LanguageClient,
+        // eslint-disable-next-line @typescript-eslint/no-empty-function
     ): void {}
 
     public dispose(): void {

test/features/DebugSession.test.ts

@@ -376,9 +376,9 @@ describe("DebugSessionFeature", () => {
                 sessionManager: sessionManager,
             });
 
-            // eslint-disable-next-line @typescript-eslint/no-explicit-any
             const pickPSHostProcessStub = Sinon.stub(
                 debugSessionFeatureStub,
+                // eslint-disable-next-line @typescript-eslint/no-explicit-any
                 "pickPSHostProcess" as any,
             ).resolves(7357);
 
@@ -408,9 +408,9 @@ describe("DebugSessionFeature", () => {
                 sessionManager: sessionManager,
             });
 
-            // eslint-disable-next-line @typescript-eslint/no-explicit-any
             const pickPSHostProcessStub = Sinon.stub(
                 debugSessionFeatureStub,
+                // eslint-disable-next-line @typescript-eslint/no-explicit-any
                 "pickPSHostProcess" as any,
             ).resolves(undefined);
 
@@ -439,9 +439,9 @@ describe("DebugSessionFeature", () => {
                 sessionManager: sessionManager,
             });
 
-            // eslint-disable-next-line @typescript-eslint/no-explicit-any
             const pickRunspaceStub = Sinon.stub(
                 debugSessionFeatureStub,
+                // eslint-disable-next-line @typescript-eslint/no-explicit-any
                 "pickRunspace" as any,
             ).resolves(TEST_NUMBER);
 
@@ -470,9 +470,9 @@ describe("DebugSessionFeature", () => {
                 sessionManager: sessionManager,
             });
 
-            // eslint-disable-next-line @typescript-eslint/no-explicit-any
             const pickRunspaceStub = Sinon.stub(
                 debugSessionFeatureStub,
+                // eslint-disable-next-line @typescript-eslint/no-explicit-any
                 "pickRunspace" as any,
             ).resolves(undefined);
 
@@ -599,9 +599,9 @@ describe("DebugSessionFeature", () => {
             attachConfig.dotnetDebuggerConfigName = foundDotnetConfig.name;
             const debugSessionFeature = createDebugSessionFeatureStub({});
 
-            // eslint-disable-next-line @typescript-eslint/no-explicit-any
             Sinon.stub(
                 debugSessionFeature,
+                // eslint-disable-next-line @typescript-eslint/no-explicit-any
                 "getLaunchConfigurations" as any,
             ).returns(candidateDotnetConfigs);