Skip to content

Unify uninstall confirmation flag help#379

Closed
GreyforgeLabs wants to merge 1 commit into
CortexLM:mainfrom
GreyforgeLabs:fix-uninstall-confirmation-help
Closed

Unify uninstall confirmation flag help#379
GreyforgeLabs wants to merge 1 commit into
CortexLM:mainfrom
GreyforgeLabs:fix-uninstall-confirmation-help

Conversation

@GreyforgeLabs
Copy link
Copy Markdown

Summary

  • Collapse uninstall confirmation skipping onto the --yes/-y field with --force/-f accepted as aliases.
  • Remove the duplicate independent --force help entry while preserving compatibility for existing commands.

Fixes PlatformNetwork/bounty-challenge#53135

Validation

  • cargo fmt --package cortex-cli --check
  • git diff --check
  • cargo check -p cortex-cli --no-default-features --features cortex-tui
  • cargo run -p cortex-cli --no-default-features --features cortex-tui --bin Cortex -- uninstall --help
  • target/debug/Cortex uninstall -f --dry-run
  • target/debug/Cortex uninstall --yes --dry-run

Copy link
Copy Markdown

@greptile-apps greptile-apps Bot left a comment

Choose a reason for hiding this comment

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

Your free trial has ended. If you'd like to continue receiving code reviews, you can add a payment method here.

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.

[BUG] [v0.0.7] cortex uninstall --force and --yes flags are functionally identical but documented as different concepts

1 participant