Skip to content

Convert remaining subcommands#22281

Open
MikeMcQuaid wants to merge 2 commits into
subcommand-parser-supportfrom
other-subcommands
Open

Convert remaining subcommands#22281
MikeMcQuaid wants to merge 2 commits into
subcommand-parser-supportfrom
other-subcommands

Conversation

@MikeMcQuaid
Copy link
Copy Markdown
Member

@MikeMcQuaid MikeMcQuaid commented May 14, 2026

  • Small state/action commands should use the same parser API.
  • Keep the legacy analytics UUID command visible but marked for removal.
  • Refresh generated manpages and completions after all conversions.
  • Have you followed the guidelines in our Contributing document?
  • Have you checked to ensure there aren't other open Pull Requests for the same change?
  • Have you added an explanation of what your changes do and why you'd like us to include them? Performance claims (e.g. "this is faster") must include Hyperfine benchmarks.
  • Have you written new tests (excluding integration tests) for your changes? Here's an example.
  • Have you successfully run brew lgtm (style, typechecking and tests) with your changes locally?
  • AI was used to generate or assist with generating this PR. Please specify below how you used AI to help you, and what steps you have taken to manually verify the changes. Non-maintainers may only have one AI-assisted/generated PR open at a time.

OpenAI Codex 5.5 xhigh with local review and testing.

@MikeMcQuaid
Add parser subcommand scaffolding
00b6e04 
- Homebrew commands need parser-owned nested actions for help and
  completions.
- Keep this first step behaviour-neutral for existing commands.
- Add focused tests for subcommand args, option scoping and metadata.
@MikeMcQuaid MikeMcQuaid force-pushed the subcommand-parser-support branch from 7ae6c50 to 00b6e04 Compare May 14, 2026 14:00
@MikeMcQuaid
Convert remaining subcommands
c8f491c 
- Small state/action commands should use the same parser API.
- Keep the legacy analytics UUID command visible but marked for removal.
- Refresh generated manpages and completions after all conversions.
@MikeMcQuaid MikeMcQuaid force-pushed the subcommand-parser-support branch from 00b6e04 to ef0af09 Compare May 14, 2026 16:05
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

No reviews

Assignees

No one assigned

Labels

None yet

Projects

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

1 participant

@MikeMcQuaid