remove the destination only if it exists (#62)

- should close #61 ## Description `nupm install --force` should not try to `rm` the content of `$destination` if it does not exist. this PR adds a check for that and a test to the CI.
nushell · Jan 28, 2024 · 8fcec7a · 8fcec7a
1 parent 66e2eaa
commit 8fcec7a
Show file tree

Hide file tree

Showing 2 changed files with 6 additions and 2 deletions.
diff --git a/.github/workflows/ci.yml b/.github/workflows/ci.yml
@@ -34,4 +34,6 @@ jobs:
         run: version
 
       - name: Run the tests
-        run: nu --commands $"use ($env.PWD)/nupm/; nupm test"
+        run: |
+          nu --commands $"use ($env.PWD)/nupm/; nupm install --no-confirm --force --path ."
+          nu --commands $"use ($env.PWD)/nupm/; nupm test"
diff --git a/nupm/install.nu b/nupm/install.nu
@@ -87,7 +87,9 @@ def install-path [
             let destination = $module_dir | path join $package.name
 
             if $force {
-                rm --recursive --force $destination
+                if ($destination | path exists) {
+                    rm --recursive --force $destination
+                }
             }
 
             if ($destination | path type) == dir {