Skip to content

Commit

Permalink
rust-remove: user-facing message update
Browse files Browse the repository at this point in the history
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
  • Loading branch information
lXXXw authored and facebook-github-bot committed Dec 17, 2024
1 parent 1fdf05b commit 4bc43ee
Show file tree
Hide file tree
Showing 2 changed files with 2 additions and 2 deletions.
2 changes: 1 addition & 1 deletion eden/fs/cli_rs/edenfs-commands/src/remove.rs
Original file line number Diff line number Diff line change
Expand Up @@ -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()
));
Expand Down
2 changes: 1 addition & 1 deletion eden/scm/tests/test-eden-remove.t
Original file line number Diff line number Diff line change
Expand Up @@ -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
Expand Down

0 comments on commit 4bc43ee

Please sign in to comment.