From 4bc43eecb1162bd332b5db09f0d10a5cfa3948a5 Mon Sep 17 00:00:00 2001 From: Xiaowei Lu Date: Tue, 17 Dec 2024 15:58:03 -0800 Subject: [PATCH] rust-remove: user-facing message update Summary: A simple update to the messaging. Since this command is named "remove", it's better to use "remove" instead of "delete". Reviewed By: jdelliot Differential Revision: D67147280 fbshipit-source-id: 2f7fa70e92c649b34e4fef5bce6092f43997d3e5 --- eden/fs/cli_rs/edenfs-commands/src/remove.rs | 2 +- eden/scm/tests/test-eden-remove.t | 2 +- 2 files changed, 2 insertions(+), 2 deletions(-) diff --git a/eden/fs/cli_rs/edenfs-commands/src/remove.rs b/eden/fs/cli_rs/edenfs-commands/src/remove.rs index 63369361275dc..64424e93dea35 100644 --- a/eden/fs/cli_rs/edenfs-commands/src/remove.rs +++ b/eden/fs/cli_rs/edenfs-commands/src/remove.rs @@ -141,7 +141,7 @@ impl Determination { if path_copy.pop() { if self.is_active_eden_mount(&path_copy) { return Err(anyhow!( - "{} is not the root of checkout {}, not deleting", + "{} is not the root of checkout {}, not removing", context, path_copy.display() )); diff --git a/eden/scm/tests/test-eden-remove.t b/eden/scm/tests/test-eden-remove.t index c2cf55cf957db..34e7c4670e991 100644 --- a/eden/scm/tests/test-eden-remove.t +++ b/eden/scm/tests/test-eden-remove.t @@ -74,7 +74,7 @@ create a test directory inside eden eden remove this directory should just give error since it's not the root dir $ EDENFSCTL_ONLY_RUST=true eden remove -q -y $TESTTMP/wcrepo/test_dir - Error: $TESTTMP/wcrepo/test_dir is not the root of checkout $TESTTMP/wcrepo, not deleting + Error: $TESTTMP/wcrepo/test_dir is not the root of checkout $TESTTMP/wcrepo, not removing [1] eden list should give two repos