Skip to content

CLI: Hide the --state-dir flag - #1503

Merged
roosterfish merged 2 commits into
canonical:mainfrom
roosterfish:drop_state_dir
Aug 14, 2026
Merged

CLI: Hide the --state-dir flag#1503
roosterfish merged 2 commits into
canonical:mainfrom
roosterfish:drop_state_dir

Conversation

@roosterfish

Copy link
Copy Markdown
Contributor

Fixes #1243.

It used used by the snaps 'microcloud' wrapper command but never by the user directly.

Copilot AI left a comment

Copy link
Copy Markdown

Choose a reason for hiding this comment

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

Pull request overview

This PR updates the microcloud CLI root command to hide the --state-dir persistent flag from help output, aiming to prevent user-facing exposure of an option primarily intended for snap wrapper usage.

Changes:

  • Hide the --state-dir persistent flag from microcloud --help.
  • Add early-fail handling if hiding the flag fails.

💡 Add a code-review agent skill or configure MCP servers for context-aware, tailored reviews. Learn more in the docs.

Comment thread cmd/microcloud/main.go
Comment thread cmd/microcloud/main.go Outdated
It used used by the snaps 'microcloud' wrapper command but never by the user directly.

Signed-off-by: Julian Pelizäus <julian.pelizaeus@canonical.com>
Signed-off-by: Julian Pelizäus <julian.pelizaeus@canonical.com>
@roosterfish
roosterfish merged commit 3c5fa4d into canonical:main Aug 14, 2026
30 of 31 checks passed
@roosterfish
roosterfish deleted the drop_state_dir branch August 14, 2026 10:52
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.

Drop --state-dir flag from CLI

3 participants