Skip to content

cmd/baton: remove Hidden flag from rollback-expansion command#962

Open
btipling wants to merge 1 commit into
mainfrom
bt/unhide-rollback-expansion
Open

cmd/baton: remove Hidden flag from rollback-expansion command#962
btipling wants to merge 1 commit into
mainfrom
bt/unhide-rollback-expansion

Conversation

@btipling

Copy link
Copy Markdown
Contributor

The rollback-expansion subcommand was registered with Hidden: true in
cmd/baton/rollback_expansion.go, which suppressed it from appearing in
baton --help output. The command is compiled in and fully functional;
hiding it makes it undiscoverable to users.

This change removes the Hidden: true field so the command appears in
the standard help listing alongside other subcommands.


Built with stargate.

The rollback-expansion subcommand was registered with Hidden: true,
making it invisible in baton --help output despite being compiled in
and fully functional. Remove the Hidden field so the command is
discoverable via help.

Co-authored-by: c1-squire-dev[bot] <c1-squire-dev[bot]@users.noreply.github.com>
@btipling btipling requested a review from a team June 15, 2026 21:49
@github-actions

Copy link
Copy Markdown
Contributor

General PR Review: cmd/baton: remove Hidden flag from rollback-expansion command

Blocking Issues: 0 | Suggestions: 0 | Threads Resolved: 0
Review mode: full
View review run

Review Summary

This PR removes the Hidden: true field from the rollback-expansion Cobra command in cmd/baton/rollback_expansion.go, so the fully-functional command appears in baton --help. The change is a single cosmetic field removal with no logic, security, or compatibility impact. No new issues found.

Security Issues

None found.

Correctness Issues

None found.

Suggestions

None.

@github-actions github-actions Bot 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.

No blocking issues found.

@ggreer ggreer 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.

Maybe in the help, mention that this should only be used if you know what you're doing. Is there any use case where a customer would want to undo grant expansion on a c1z?

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.

2 participants