holm/nushell
1
1
Fork 0
forked from extern/nushell
Code Issues Pull Requests Packages Projects Releases Wiki Activity

Make default completion mode OS dependent (#1636)

Browse Source
This commit is contained in:
Pirmin Kalberer 2020-04-23 00:53:40 +02:00 committed by GitHub
parent b3c623396f
commit 818171cc2c
No known key found for this signature in database
GPG Key ID: 4AEE18F83AFDEB23
1 changed files with 7 additions and 2 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

9
crates/nu-cli/src/cli.rs
View File

@ -469,6 +469,11 @@ pub async fn run_pipeline_standalone(
/// The entry point for the CLI. Will register all known internal commands, load experimental commands, load plugins, then prepare the prompt and line reader for input.
pub async fn cli() -> Result<(), Box<dyn Error>> {
#[cfg(windows)]
const DEFAULT_COMPLETION_MODE: CompletionType = CompletionType::Circular;
#[cfg(not(windows))]
const DEFAULT_COMPLETION_MODE: CompletionType = CompletionType::List;
let mut syncer = crate::env::environment_syncer::EnvironmentSyncer::new();
let mut context = create_default_context(&mut syncer)?;
@ -555,9 +560,9 @@ pub async fn cli() -> Result<(), Box<dyn Error>> {
.map(|s| match s.value.expect_string() {
"list" => CompletionType::List,
"circular" => CompletionType::Circular,
_ => CompletionType::Circular,
_ => DEFAULT_COMPLETION_MODE,
})
.unwrap_or(CompletionType::Circular);
.unwrap_or(DEFAULT_COMPLETION_MODE);
rl.set_completion_type(completion_mode);