Avoid claiming we would move something even though we won't (in 'organize')

The check to see if a repository is already in place was done only in
'execute' mode.

Author: Byron

gitoxide-core/src/organize.rs

@@ -154,23 +154,24 @@ fn handle(
                 .ok_or_else(|| anyhow::Error::msg(format!("Remote URLs must have host names: {}", url)))?,
         )
         .join(to_relative(git_url::expand_path(None, url.path.as_bstr())?));
+
+    if git_workdir.canonicalize()? == destination.canonicalize()? {
+        progress.info(format!(
+            "Skipping {:?} as it is in the correct spot",
+            git_workdir.display()
+        ));
+        return Ok(());
+    }
     match mode {
         Mode::Simulate => progress.info(format!(
             "WOULD move {} to {}",
             git_workdir.display(),
             destination.display()
         )),
         Mode::Execute => {
-            if git_workdir.canonicalize()? == destination {
-                progress.info(format!(
-                    "Skipping {:?} as it is in the correct spot",
-                    git_workdir.display()
-                ));
-            } else {
-                std::fs::create_dir_all(destination.parent().expect("repo destination is not the root"))?;
-                progress.done(format!("Moving {} to {}", git_workdir.display(), destination.display()));
-                std::fs::rename(git_workdir, &destination)?;
-            }
+            std::fs::create_dir_all(destination.parent().expect("repo destination is not the root"))?;
+            progress.done(format!("Moving {} to {}", git_workdir.display(), destination.display()));
+            std::fs::rename(git_workdir, &destination)?;
         }
     }
     Ok(())