From de7c40789ef082fbf13f4a6cdaa504c95c176fe9 Mon Sep 17 00:00:00 2001 From: Azeem Sajid Date: Wed, 29 Jan 2025 08:22:32 +0500 Subject: [PATCH] Use `defer` instead --- pkg/cmd/run/list/list.go | 3 ++- 1 file changed, 2 insertions(+), 1 deletion(-) diff --git a/pkg/cmd/run/list/list.go b/pkg/cmd/run/list/list.go index 0b291fae5..3113fdabd 100644 --- a/pkg/cmd/run/list/list.go +++ b/pkg/cmd/run/list/list.go @@ -119,6 +119,8 @@ func listRun(opts *ListOptions) error { } opts.IO.StartProgressIndicator() + defer opts.IO.StopProgressIndicator() + if opts.WorkflowSelector != "" { // initially the workflow state is limited to 'active' states := []workflowShared.WorkflowState{workflowShared.Active} @@ -131,7 +133,6 @@ func listRun(opts *ListOptions) error { filters.WorkflowID = workflow.ID filters.WorkflowName = workflow.Name } else { - opts.IO.StopProgressIndicator() return err } }