Normalize MarkDeprecated() messages

What does this MR do and why?

Normalizes the 14 MarkDeprecated() calls across the codebase to use a consistent style. Cobra renders these as:

Flag --X has been deprecated, <message>

so the message reads most naturally with a lowercase first letter and a trailing period, treated as a sentence continuation. Eight of the existing 14 messages already follow this style; this MR brings the remaining six in line.

Changes (6 files):

  • mr/checkout/mr_checkout.goNow enabled by defaultnow enabled by default.
  • mr/create/mr_create.goUse --repo instead.use --repo instead.
  • ci/get/get.goDeprecated. Use 'output' instead.use --output instead. (drops the redundant "Deprecated." prefix since cobra already says "has been deprecated,")
  • ci/list/list.gouse --order insteaduse --order instead. (adds period)
  • release/list/release_list.goUse \glab release view ` instead.` → lowercases first letter
  • issuable/list/issuable_list.gouse --assignee=@meuse --assignee=@me.

How to verify

make lint           # 0 issues
make gen-docs       # no changes — these messages don't appear in generated flag help
glab mr checkout --track 1   # see the new deprecation warning

Author's checklist

  • make lint passes with 0 issues.
  • All 14 MarkDeprecated() calls now follow the same style.
  • No semantic change; pure presentation.
Edited by Mukunda Katta

Merge request reports

Loading