refactor: remove nesting during dry-run of argo delete (#11596)

Signed-off-by: Anton Gilgur <[email protected]>
argoproj · Aug 17, 2023 · bda561e · bda561e
1 parent 1cc53f9
commit bda561e
Showing 1 changed file with 10 additions and 9 deletions.
diff --git a/cmd/argo/commands/delete.go b/cmd/argo/commands/delete.go
@@ -64,17 +64,18 @@ func NewDeleteCommand() *cobra.Command {
  }
 
  for _, wf := range workflows {
- if !dryRun {
- _, err := serviceClient.DeleteWorkflow(ctx, &workflowpkg.WorkflowDeleteRequest{Name: wf.Name, Namespace: wf.Namespace, Force: force})
- if err != nil && status.Code(err) == codes.NotFound {
- fmt.Printf("Workflow '%s' not found\n", wf.Name)
- continue
- }
- errors.CheckError(err)
- fmt.Printf("Workflow '%s' deleted\n", wf.Name)
- } else {
+ if dryRun {
  fmt.Printf("Workflow '%s' deleted (dry-run)\n", wf.Name)
+ continue
  }
+
+ _, err := serviceClient.DeleteWorkflow(ctx, &workflowpkg.WorkflowDeleteRequest{Name: wf.Name, Namespace: wf.Namespace, Force: force})
+ if err != nil && status.Code(err) == codes.NotFound {
+ fmt.Printf("Workflow '%s' not found\n", wf.Name)
+ continue
+ }
+ errors.CheckError(err)
+ fmt.Printf("Workflow '%s' deleted\n", wf.Name)
  }
  },
  }