cli/edit: Don't error out if host spec unchanged

jlebon · jlebon · commit 0f166638e929 · 2023-11-01T09:32:50.000-04:00
This matches e.g. `kubectl edit`.

Signed-off-by: Jonathan Lebon &lt;jonathan@jlebon.com&gt;
diff --git a/lib/src/cli.rs b/lib/src/cli.rs
@@ -438,7 +438,8 @@ async fn edit(opts: EditOpts) -> Result<()> {
     };
 
     if new_host.spec == host.spec {
-        anyhow::bail!("No changes in current host spec");
+        println!("Edit cancelled, no changes made.");
+        return Ok(());
     }
     let new_spec = RequiredHostSpec::from_spec(&new_host.spec)?;
     let fetched = pull(repo, &new_spec.image, opts.quiet).await?;

Original file line number	Diff line number	Diff line change
`@@ -438,7 +438,8 @@ async fn edit(opts: EditOpts) -> Result<()> {`
`438`	`438`	`};`
`439`	`439`
`440`	`440`	`if new_host.spec == host.spec {`
`441`		`- anyhow::bail!("No changes in current host spec");`
	`441`	`+ println!("Edit cancelled, no changes made.");`
	`442`	`+ return Ok(());`
`442`	`443`	`}`
`443`	`444`	`let new_spec = RequiredHostSpec::from_spec(&new_host.spec)?;`
`444`	`445`	`let fetched = pull(repo, &new_spec.image, opts.quiet).await?;`