Skip to content

Fix CLI/docs consistency: document --shellcheck and --version, normalize update flag help - #50280

Merged
pelikhan merged 2 commits into
mainfrom
copilot/cli-consistency-fixes
Aug 4, 2026
Merged

Fix CLI/docs consistency: document --shellcheck and --version, normalize update flag help#50280
pelikhan merged 2 commits into
mainfrom
copilot/cli-consistency-fixes

Conversation

Copilot AI commented Aug 4, 2026

Copy link
Copy Markdown
Contributor

The CLI consistency audit found gh aw compile --shellcheck and the global --version flag missing from the docs, plus inconsistent help wording on gh aw update's negation flags.

Docs (docs/src/content/docs/setup/cli.md)

  • Added --shellcheck to the compile Options summary.
  • Added a --version row to the Global Options table (previously only the version subcommand was mentioned).

gh aw update help text (pkg/cli/update_command.go)

  • --force: now scoped explicitly — "Force update of workflow files even if no changes are detected".
  • --no-merge, --no-redirect, --no-release-bump (and deprecated --disable-release-bump) rewritten to the "Skip X" phrasing used by the rest of the CLI:
--no-merge          Skip merging; override local changes with the upstream version
--no-redirect       Skip following redirects; refuse updates when redirect frontmatter is present
--no-release-bump   Skip automatic major version bumps for non-core actions (only core actions/* are bumped)

Notes on findings not applied

  • --pyflakes is only registered on lint, not compile, so it was intentionally left out of compile's Options list.
  • Renaming update --force to --force-update would be a breaking CLI change; the clarified description addresses the ambiguity without breaking existing invocations.

Co-authored-by: pelikhan <4175913+pelikhan@users.noreply.github.com>
Copilot AI changed the title [WIP] Fix inconsistencies in CLI help output Fix CLI/docs consistency: document --shellcheck and --version, normalize update flag help Aug 4, 2026
Copilot AI requested a review from pelikhan August 4, 2026 14:43
@pelikhan
pelikhan marked this pull request as ready for review August 4, 2026 14:53
Copilot AI balanced review requested due to automatic review settings August 4, 2026 14:53
@pelikhan
pelikhan merged commit 899e0b5 into main Aug 4, 2026
1 check passed
@pelikhan
pelikhan deleted the copilot/cli-consistency-fixes branch August 4, 2026 14:53

Copilot AI left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Pull request overview

Aligns CLI documentation and update help text with actual command options.

Changes:

  • Documents global --version and compile --shellcheck.
  • Clarifies --force and normalizes --no-* help wording.
Show a summary per file
File Description
docs/src/content/docs/setup/cli.md Documents missing CLI flags.
pkg/cli/update_command.go Revises update flag descriptions.

Review details

Tip

Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.

  • Files reviewed: 2/2 changed files
  • Comments generated: 1
  • Review effort level: Balanced

Comment thread pkg/cli/update_command.go
Comment on lines +171 to +172
cmd.Flags().Bool("no-release-bump", false, "Skip automatic major version bumps for non-core actions (only core actions/* are bumped)")
cmd.Flags().Bool("disable-release-bump", false, "Skip automatic major version bumps for non-core actions (only core actions/* are bumped)")
@github-actions

github-actions Bot commented Aug 4, 2026

Copy link
Copy Markdown
Contributor

🎉 This pull request is included in a new release.

Release: v0.84.4

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

[cli-consistency] CLI Consistency Issues - 2026-08-04

3 participants