Skip to content

Commit c101e00

Browse files
committed
Always strip "default" option from help description
1 parent d4350b7 commit c101e00

File tree

2 files changed

+7
-8
lines changed

2 files changed

+7
-8
lines changed

Rules.md

Lines changed: 2 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -3898,10 +3898,10 @@ Align wrapped function arguments or collection elements.
38983898
Option | Description
38993899
--- | ---
39003900
`--wrap-arguments` | Function argument wrapping: "before-first", "after-first", "preserve" (default) or "disabled"
3901-
`--wrap-parameters` | Function call parameter wrapping: "before-first", "after-first", "preserve", "disabled" or "default"
3901+
`--wrap-parameters` | Function call parameter wrapping: "before-first", "after-first", "preserve" or "disabled"
39023902
`--wrap-collections` | Collection literal element wrapping: "before-first", "after-first", "preserve" (default) or "disabled"
39033903
`--closing-paren` | Closing paren placement: "balanced" (default) or "same-line"
3904-
`--call-site-paren` | Closing paren placement at function call sites: "balanced", "same-line" or "default"
3904+
`--call-site-paren` | Closing paren placement at function call sites: "balanced" or "same-line"
39053905
`--wrap-return-type` | Function return type wrapping: "preserve" (default), "if-multiline" or "never"
39063906
`--wrap-conditions` | Conditional expression wrapping: "before-first", "after-first", "preserve" (default) or "disabled"
39073907
`--wrap-type-aliases` | Typealias wrapping: "before-first", "after-first", "preserve" (default) or "disabled"

Sources/Arguments.swift

Lines changed: 5 additions & 6 deletions
Original file line numberDiff line numberDiff line change
@@ -83,13 +83,12 @@ extension Array where Element: Equatable {
8383
assert(contains(where: { $0 == `default` }))
8484
}
8585
let options: [String] = compactMap {
86-
switch ("\($0)" == "default", $0 == `default`) {
87-
case (true, true), (false, false):
88-
return "\"\($0)\""
89-
case (false, true):
86+
if "\($0)" == "default" {
87+
return nil
88+
} else if $0 == `default` {
9089
return "\"\($0)\" (default)"
91-
case (true, false):
92-
return nil // Don't show "default" if it's not the default
90+
} else {
91+
return "\"\($0)\""
9392
}
9493
}
9594
return options.formattedList(lastSeparator: "or")

0 commit comments

Comments
 (0)