Merge pull request #1469 from AkihiroSuda/edit-fail

AkihiroSuda · web-flow · commit 9a5ca0db061c · 2023-04-09T19:56:24.000+09:00
limactl: let `limactl edit non-existent-instance` fail
diff --git a/cmd/limactl/edit.go b/cmd/limactl/edit.go
@@ -43,8 +43,7 @@ func editAction(cmd *cobra.Command, args []string) error {
 	inst, err := store.Inspect(instName)
 	if err != nil {
 		if errors.Is(err, os.ErrNotExist) {
-			logrus.Infof("Instance %q not found", instName)
-			return nil
+			return fmt.Errorf("Instance %q not found", instName)
 		}
 		return err
 	}

Original file line number	Diff line number	Diff line change
`@@ -43,8 +43,7 @@ func editAction(cmd *cobra.Command, args []string) error {`
`43`	`43`	`inst, err := store.Inspect(instName)`
`44`	`44`	`if err != nil {`
`45`	`45`	`if errors.Is(err, os.ErrNotExist) {`
`46`		`- logrus.Infof("Instance %q not found", instName)`
`47`		`- return nil`
	`46`	`+ return fmt.Errorf("Instance %q not found", instName)`
`48`	`47`	`}`
`49`	`48`	`return err`
`50`	`49`	`}`