Skip to content

Commit bd6580b

Browse files
completer: drop now-wrong ln example from complete_option comment
ln -s is a boolean flag after the merged spec fix, so it is no longer an option with differing-typed arguments. Cite jq --rawfile instead.
1 parent 6f3b9b1 commit bd6580b

1 file changed

Lines changed: 1 addition & 1 deletion

File tree

  • crates/warp_completer/src/completer/engine/argument

crates/warp_completer/src/completer/engine/argument/legacy.rs

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -441,7 +441,7 @@ async fn complete_option(
441441
) -> Vec<MatchedSuggestion> {
442442
// Resolve the option's argument at the position of the value being completed, mirroring
443443
// `complete_positional`, instead of always taking the last declared argument. An option can
444-
// declare several arguments of differing types (e.g. `ln -s <source_file> <link_name>`), so
444+
// declare several arguments of differing types (e.g. `jq --rawfile <name> <file>`), so
445445
// `.last()` offers the wrong argument's completions for any value but the last. A variadic
446446
// argument at or before this index absorbs the value, since a variadic argument need not be the
447447
// last one declared.

0 commit comments

Comments
 (0)