chore: return nil as err already handled

DanielleMaywood · DanielleMaywood · commit f2ed4fa10591 · 2024-09-04T10:18:50.000Z
diff --git a/cli/cliui/select.go b/cli/cliui/select.go
@@ -96,7 +96,6 @@ func Select(inv *serpent.Invocation, opts SelectOptions) (string, error) {
 		tea.WithInput(inv.Stdin),
 		tea.WithOutput(inv.Stdout),
 	).Run()
-
 	if err != nil {
 		return "", err
 	}
@@ -110,7 +109,7 @@ func Select(inv *serpent.Invocation, opts SelectOptions) (string, error) {
 		return "", Canceled
 	}
 
-	return model.selected, err
+	return model.selected, nil
 }
 
 type selectModel struct {
@@ -297,7 +296,6 @@ func MultiSelect(inv *serpent.Invocation, opts MultiSelectOptions) ([]string, er
 		tea.WithInput(inv.Stdin),
 		tea.WithOutput(inv.Stdout),
 	).Run()
-
 	if err != nil {
 		return nil, err
 	}
@@ -311,7 +309,7 @@ func MultiSelect(inv *serpent.Invocation, opts MultiSelectOptions) ([]string, er
 		return nil, Canceled
 	}
 
-	return model.selectedOptions(), err
+	return model.selectedOptions(), nil
 }
 
 type multiSelectOption struct {

Original file line number	Diff line number	Diff line change
`@@ -96,7 +96,6 @@ func Select(inv *serpent.Invocation, opts SelectOptions) (string, error) {`
`96`	`96`	`tea.WithInput(inv.Stdin),`
`97`	`97`	`tea.WithOutput(inv.Stdout),`
`98`	`98`	`).Run()`
`99`		`-`
`100`	`99`	`if err != nil {`
`101`	`100`	`return "", err`
`102`	`101`	`}`
`@@ -110,7 +109,7 @@ func Select(inv *serpent.Invocation, opts SelectOptions) (string, error) {`
`110`	`109`	`return "", Canceled`
`111`	`110`	`}`
`112`	`111`
`113`		`- return model.selected, err`
	`112`	`+ return model.selected, nil`
`114`	`113`	`}`
`115`	`114`
`116`	`115`	`type selectModel struct {`
`@@ -297,7 +296,6 @@ func MultiSelect(inv *serpent.Invocation, opts MultiSelectOptions) ([]string, er`
`297`	`296`	`tea.WithInput(inv.Stdin),`
`298`	`297`	`tea.WithOutput(inv.Stdout),`
`299`	`298`	`).Run()`
`300`		`-`
`301`	`299`	`if err != nil {`
`302`	`300`	`return nil, err`
`303`	`301`	`}`
`@@ -311,7 +309,7 @@ func MultiSelect(inv *serpent.Invocation, opts MultiSelectOptions) ([]string, er`
`311`	`309`	`return nil, Canceled`
`312`	`310`	`}`
`313`	`311`
`314`		`- return model.selectedOptions(), err`
	`312`	`+ return model.selectedOptions(), nil`
`315`	`313`	`}`
`316`	`314`
`317`	`315`	`type multiSelectOption struct {`