From 77972af03f4206b2a79ec5d9c4c966e422bd4977 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 c4f24ccbbd..6bdfcbf2c8 100644 --- a/cli/src/commands/mod.rs +++ b/cli/src/commands/mod.rs @@ -800,10 +800,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, } @@ -3067,7 +3071,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: {}