From d6ebe1c8124c9536b197913a051004e5110afa81 Mon Sep 17 00:00:00 2001 From: Waleed Khan Date: Tue, 22 Aug 2023 08:43:09 -0700 Subject: [PATCH] commands: add `--interactive` flag to `jj split` I am constantly explicitly running `jj split -i` because I expect the interactive flag to be there for consistency. --- cli/src/commands/mod.rs | 8 ++++++-- 1 file changed, 6 insertions(+), 2 deletions(-) diff --git a/cli/src/commands/mod.rs b/cli/src/commands/mod.rs index 7995931ea1..0a455c3b9a 100644 --- a/cli/src/commands/mod.rs +++ b/cli/src/commands/mod.rs @@ -799,10 +799,14 @@ struct DiffeditArgs { /// asked for a description only for the first part. #[derive(clap::Args, Clone, Debug)] struct SplitArgs { + /// Interactively choose which parts to split. This is the default if no + /// paths are provided. + #[arg(long, short)] + interactive: bool, /// The revision to split #[arg(long, short, default_value = "@")] revision: RevisionArg, - /// Put these paths in the first commit and don't run the diff editor + /// Put these paths in the first commit #[arg(value_hint = clap::ValueHint::AnyPath)] paths: Vec, } @@ -3065,7 +3069,7 @@ fn cmd_split(ui: &mut Ui, command: &CommandHelper, args: &SplitArgs) -> Result<( let mut tx = workspace_command.start_transaction(&format!("split commit {}", commit.id().hex())); let base_tree = merge_commit_trees(tx.repo(), &commit.parents())?; - let interactive = args.paths.is_empty(); + let interactive = args.interactive || args.paths.is_empty(); let instructions = format!( "\ You are splitting a commit in two: {}