Move detection logic to the RubyInstaller class

Author: rafaelfranca

vscode/src/ruby.ts

@@ -1,5 +1,4 @@
 import path from "path";
-import os from "os";
 
 import * as vscode from "vscode";
 
@@ -328,11 +327,6 @@ export class Ruby implements RubyInterface {
       }
     }
 
-    if (os.platform() === "win32") {
-      this.versionManager = ManagerIdentifier.RubyInstaller;
-      return;
-    }
-
     // If we can't find a version manager, just return None
     this.versionManager = ManagerIdentifier.None;
   }

vscode/src/ruby/rubyInstaller.ts

@@ -16,6 +16,14 @@ interface RubyVersion {
 //
 // If we can't find it there, then we throw an error and rely on the user to manually select where Ruby is installed.
 export class RubyInstaller extends Chruby {
+  // eslint-disable-next-line @typescript-eslint/require-await
+  static async detect(
+    _workspaceFolder: vscode.WorkspaceFolder,
+    _outputChannel: vscode.LogOutputChannel,
+  ): Promise<vscode.Uri | undefined> {
+    return os.platform() === "win32" ? vscode.Uri.file("RubyInstaller") : undefined;
+  }
+
   // Environment variables are case sensitive on Windows when we access them through NodeJS. We need to ensure that
   // we're searching through common variations, so that we don't accidentally miss the path we should inherit
   protected getProcessPath() {