diff --git a/cli/src/merge_tools/mod.rs b/cli/src/merge_tools/mod.rs index 34cb37bbd..96b9dc8ea 100644 --- a/cli/src/merge_tools/mod.rs +++ b/cli/src/merge_tools/mod.rs @@ -155,6 +155,27 @@ pub enum MergeTool { External(ExternalMergeTool), } +/// Finds the appropriate tool for diff editing or merges +fn editor_args_from_settings( + ui: &Ui, + settings: &UserSettings, + key: &str, +) -> Result { + // TODO: Make this configuration have a table of possible editors and detect the + // best one here. + if let Some(args) = settings.config().get(key).optional()? { + Ok(args) + } else { + let default_editor = "meld"; + writeln!( + ui.hint(), + "Using default editor '{default_editor}'; you can change this by setting {key}" + ) + .map_err(ExternalToolError::Io)?; + Ok(default_editor.into()) + } +} + /// Loads merge tool options from `[merge-tools.]`. pub fn get_tool_config( settings: &UserSettings, @@ -216,27 +237,6 @@ fn get_merge_tool_from_settings( } } -/// Finds the appropriate tool for diff editing or merges -fn editor_args_from_settings( - ui: &Ui, - settings: &UserSettings, - key: &str, -) -> Result { - // TODO: Make this configuration have a table of possible editors and detect the - // best one here. - if let Some(args) = settings.config().get(key).optional()? { - Ok(args) - } else { - let default_editor = "meld"; - writeln!( - ui.hint(), - "Using default editor '{default_editor}'; you can change this by setting {key}" - ) - .map_err(ExternalToolError::Io)?; - Ok(default_editor.into()) - } -} - #[cfg(test)] mod tests { use super::*;