Skip to content

Commit

Permalink
cli branch move: document that we don't want to support -f for `--f…
Browse files Browse the repository at this point in the history
…rom`

Reasoning: we don't want it confused with --force
  • Loading branch information
ilyagr committed Oct 16, 2024
1 parent b795bf9 commit 6142973
Showing 1 changed file with 5 additions and 0 deletions.
5 changes: 5 additions & 0 deletions cli/src/commands/bookmark/move.rs
Original file line number Diff line number Diff line change
Expand Up @@ -42,10 +42,15 @@ use crate::ui::Ui;
#[command(group(clap::ArgGroup::new("source").multiple(true).required(true)))]
pub struct BookmarkMoveArgs {
/// Move bookmarks from the given revisions
// We intentionally do not support the short `-f` for `--from` since it
// could be confused with a shorthand for `--force`, and people might not
// realize they need `-B`/`--allow-backwards` instead.
#[arg(long, group = "source", value_name = "REVISIONS")]
from: Vec<RevisionArg>,

/// Move bookmarks to this revision
// We intentionally do not support the short `-t` for `--to` since we don't
// support `-f` for `--from`.
#[arg(long, default_value = "@", value_name = "REVISION")]
to: RevisionArg,

Expand Down

0 comments on commit 6142973

Please sign in to comment.