Skip to content

Commit 125dbe0

Browse files
authored
Merge pull request #4667 from corneliusroemer/fix-4666
fix: Add plural s to context in suggestions
2 parents 401d86f + 5dcc5c9 commit 125dbe0

File tree

2 files changed

+4
-1
lines changed

2 files changed

+4
-1
lines changed

src/error/format.rs

Lines changed: 3 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -438,6 +438,9 @@ fn did_you_mean(styled: &mut StyledStr, context: &str, valid: &ContextValue) {
438438
styled.none(TAB);
439439
styled.good("note: ");
440440
styled.none(context);
441+
if valid.len() > 1 {
442+
styled.none("s");
443+
}
441444
styled.none(" ");
442445
for (i, valid) in valid.iter().enumerate() {
443446
if i != 0 {

tests/builder/subcommands.rs

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -120,7 +120,7 @@ fn subcmd_did_you_mean_output_ambiguous() {
120120
static DYM_SUBCMD_AMBIGUOUS: &str = "\
121121
error: unrecognized subcommand 'te'
122122
123-
note: subcommand 'test', 'temp' exist
123+
note: subcommands 'test', 'temp' exist
124124
note: to pass 'te' as a value, use 'dym -- te'
125125
126126
Usage: dym [COMMAND]

0 commit comments

Comments
 (0)