From 9f81de7ef3dff7ea716056e11deaf48b1ddea0be Mon Sep 17 00:00:00 2001 From: Fintan Halpenny Date: Wed, 25 Oct 2023 14:31:41 +0100 Subject: [PATCH] cli: use `WriteStorage::remove` in `rm` Instead of calling `std::fs::remove_dir_all` in the `rad rm` command, use the `WriteStorage::remove` method. Signed-off-by: Fintan Halpenny X-Clacks-Overhead: GNU Terry Pratchett --- radicle-cli/src/commands/rm.rs | 4 ++-- 1 file changed, 2 insertions(+), 2 deletions(-) diff --git a/radicle-cli/src/commands/rm.rs b/radicle-cli/src/commands/rm.rs index 9b214cc0..2675db04 100644 --- a/radicle-cli/src/commands/rm.rs +++ b/radicle-cli/src/commands/rm.rs @@ -1,5 +1,4 @@ use std::ffi::OsString; -use std::fs; use anyhow::anyhow; @@ -7,6 +6,7 @@ use radicle::identity::Id; use radicle::node; use radicle::node::Handle as _; use radicle::storage; +use radicle::storage::WriteStorage; use radicle::Profile; use crate::git; @@ -82,7 +82,7 @@ pub fn run(options: Options, ctx: impl term::Context) -> anyhow::Result<()> { if !options.confirm || term::confirm(format!("Remove {rid}?")) { untrack(&rid, &profile)?; remove_remote(&rid)?; - fs::remove_dir_all(path)?; + storage.remove(rid)?; term::success!("Successfully removed {rid} from storage"); }