mab122
/
radicle-heartwood-lfs
1
0
Fork 0
Code Issues Pull Requests Actions Packages Projects Releases Wiki Activity

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 <fintan.halpenny@gmail.com>
X-Clacks-Overhead: GNU Terry Pratchett
This commit is contained in:
Fintan Halpenny 2023-10-25 14:31:41 +01:00 committed by cloudhead
parent 703a6d8025
commit 9f81de7ef3
No known key found for this signature in database
1 changed files with 2 additions and 2 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

4
radicle-cli/src/commands/rm.rs
View File

@ -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");
}