Skip to content

Rename Available Commands to Main Commands#4975

Open
janniklasrose wants to merge 3 commits intomainfrom
janniklasrose/rename-available-commands
Open

Rename Available Commands to Main Commands#4975
janniklasrose wants to merge 3 commits intomainfrom
janniklasrose/rename-available-commands

Conversation

@janniklasrose
Copy link
Copy Markdown
Contributor

Changes

Rename Available Commands to Main Commands

Why

If Management Commands and/or Permission Commands groups are present, it is a bit confusing from --help that they are also available.

before:

% databricks permissions --help
Usage:
  databricks permissions [flags]
  databricks permissions [command]

Available Commands
  get                   Get object permissions.
  set                   Set object permissions.
  update                Update object permissions.

Permission Commands
  get-permission-levels Get object permission levels.

after:

% ./cli permissions --help
Usage:
  databricks permissions [flags]
  databricks permissions [command]

Main Commands
  get                   Get object permissions.
  set                   Set object permissions.
  update                Update object permissions.

Permission Commands
  get-permission-levels Get object permission levels.

Flags:
  -h, --help   help for permissions

Tests

manual, see Why section above ^

@janniklasrose janniklasrose requested a review from pietern April 15, 2026 09:57
@janniklasrose janniklasrose marked this pull request as ready for review April 15, 2026 10:58
Comment thread cmd/cmd.go
{
ID: mainGroup,
Title: "Available Commands",
Title: "Main Commands",
Copy link
Copy Markdown
Contributor Author

Choose a reason for hiding this comment

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

This is now a bit inconsistent with how Cobra behaves if there are no groups (it still prints Available Commands: in that case).

We could also do "Available Commands (Main)", "Available Commands (Permissions)", here

@github-actions
Copy link
Copy Markdown

Approval status: pending

/acceptance/pipelines/ - needs approval

Files: acceptance/pipelines/databricks-cli-help/output.txt
Suggested: @kanterov
Also eligible: @lennartkats-db, @jefferycheng1

General files (require maintainer)

Files: acceptance/cmd/unknown-subcommand/output.txt, acceptance/cmd/workspace/secrets/output.txt, cmd/cmd.go
Based on git history:

  • @pietern -- recent work in cmd/

Any maintainer (@andrewnester, @anton-107, @denik, @pietern, @shreyas-goenka, @simonfaltum, @renaudhartert-db) can approve all areas.
See OWNERS for ownership rules.

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.

1 participant