Make use of registerIDFCommand

- Renaming the function to clearly indicate it's async
espressif · Jan 16, 2025 · 87753b6 · 87753b6
1 parent 7570f35
commit 87753b6
Show file tree

Hide file tree

Showing 2 changed files with 3 additions and 8 deletions.
diff --git a/src/extension.ts b/src/extension.ts
@@ -173,7 +173,7 @@ import {
   IDFWebCommandKeys,
 } from "./cmdTreeView/cmdStore";
 import { IdfSetup } from "./views/setup/types";
-import { removeEspIdfSettings } from "./uninstall";
+import { asyncRemoveEspIdfSettings } from "./uninstall";
 
 // Global variables shared by commands
 let workspaceRoot: vscode.Uri;
@@ -3731,12 +3731,7 @@ export async function activate(context: vscode.ExtensionContext) {
   checkAndNotifyMissingCompileCommands();
 
   // Remove ESP-IDF settings
-  context.subscriptions.push(
-    vscode.commands.registerCommand(
-      "espIdf.removeEspIdfSettings",
-      removeEspIdfSettings
-    )
-  );
+  registerIDFCommand("espIdf.removeEspIdfSettings", asyncRemoveEspIdfSettings);
 }
 
 function checkAndNotifyMissingCompileCommands() {

diff --git a/src/uninstall.ts b/src/uninstall.ts
@@ -2,7 +2,7 @@ import * as vscode from "vscode";
 import { Logger } from "./logger/logger";
 import { OutputChannel } from "./logger/outputChannel";
 
-export async function removeEspIdfSettings() {
+export async function asyncRemoveEspIdfSettings() {
   const config = vscode.workspace.getConfiguration();
   const settingsToDelete: string[] = [];