docs: scope the boolean flags FAQ to pre-v2.36.0 CLIs - #347
Conversation
CLI v2.36.0 accepts boolean flag values written with a space (kosli-dev/cli#1037), so the page's central claim, that an explicitly specified boolean value must use an "=", and its worked example were both wrong for anyone on a current CLI. Rewritten rather than removed, and the "Boolean flags" heading kept verbatim. CLI versions up to v2.35.0 still hardcode a link to the #boolean-flags anchor in the error they print, and those customers are exactly the ones the old explanation still serves, so the anchor has to keep resolving and the pre-v2.36.0 behaviour has to stay documented. The example now uses --compliant false, the case where the old parse silently inverted the caller's intent, instead of --compliant true where the stray argument changed nothing.
|
Claude finished @JonJagger's task in 1m 1s —— View job PR Review: Scope boolean flags FAQ to pre-v2.36.0 CLIs
Scope: FindingsNo critical or improvement issues found. This is a clean, well-scoped edit. Suggestions:
Checks
SummaryGood change. The rewrite correctly frames the space-separator limitation as version-specific (pre-v2.36.0) rather than absolute, while preserving the |
|
Preview deployment for your docs. Learn more about Mintlify Previews.
|
|
Preview deployment for your docs. Learn more about Mintlify Previews.
|
CLI v2.36.0 accepts boolean flag values written with a space
(kosli-dev/cli#1037), so the page's central claim, that an explicitly
specified boolean value must use an "=", and its worked example were both
wrong for anyone on a current CLI.
Rewritten rather than removed, and the "Boolean flags" heading kept
verbatim. CLI versions up to v2.35.0 still hardcode a link to the
#boolean-flags anchor in the error they print, and those customers are
exactly the ones the old explanation still serves, so the anchor has to
keep resolving and the pre-v2.36.0 behaviour has to stay documented.
The example now uses --compliant false, the case where the old parse
silently inverted the caller's intent, instead of --compliant true where
the stray argument changed nothing.