From ac2631f96e0c735ebb7f6c840b05db5195f6d9fc Mon Sep 17 00:00:00 2001 From: Pieter Noordhuis Date: Mon, 4 May 2026 11:43:45 +0200 Subject: [PATCH] Remove no-op promptui.SearchPrompt assignment in warehouse picker The line `promptui.SearchPrompt = "Search: "` in `SelectWarehouse` assigns the package-level global to its own default value (see `promptui/select.go`: `var SearchPrompt = "Search: "`), so it has no effect. It was introduced in #4170 (the only write to this global in the repo) alongside the warehouse picker added in #956. Drop the line and the now-unused promptui import. Co-authored-by: Isaac --- libs/databrickscfg/cfgpickers/warehouses.go | 2 -- 1 file changed, 2 deletions(-) diff --git a/libs/databrickscfg/cfgpickers/warehouses.go b/libs/databrickscfg/cfgpickers/warehouses.go index 91fdadaa91a..c08ec21d2c6 100644 --- a/libs/databrickscfg/cfgpickers/warehouses.go +++ b/libs/databrickscfg/cfgpickers/warehouses.go @@ -15,7 +15,6 @@ import ( "github.com/databricks/databricks-sdk-go/httpclient" "github.com/databricks/databricks-sdk-go/service/sql" "github.com/fatih/color" - "github.com/manifoldco/promptui" ) var ErrNoCompatibleWarehouses = errors.New("no compatible warehouses") @@ -225,7 +224,6 @@ func SelectWarehouse(ctx context.Context, w *databricks.WorkspaceClient, descrip if description != "" { cmdio.LogString(ctx, description) } - promptui.SearchPrompt = "Search: " warehouseId, err := cmdio.SelectOrdered(ctx, items, "warehouse\n") if err != nil { return "", err