Skip to content

Fix plugin uninstall command and CLI help inconsistency#649

Open
saulob wants to merge 1 commit intofatihak:mainfrom
saulob:fix/plugin-uninstall-command
Open

Fix plugin uninstall command and CLI help inconsistency#649
saulob wants to merge 1 commit intofatihak:mainfrom
saulob:fix/plugin-uninstall-command

Conversation

@saulob
Copy link
Copy Markdown

@saulob saulob commented Mar 30, 2026

This PR fixes an inconsistency between the CLI and the documentation for plugin removal

Currently:

  • The CLI help shows inkypi plugin remove <plugin_id>
  • The wiki documents inkypi plugin uninstall <plugin_id>
  • The router already supports uninstall, but the usage text and comments still reference remove

Changes:

  • Update CLI usage text from remove to uninstall
  • Update comments from REMOVE to UNINSTALL
  • Align CLI with documented command

Result:

  • CLI help now correctly reflects the actual supported command
  • Consistency between CLI and documentation

Testing:

  • Verified that inkypi plugin shows uninstall in help
  • Successfully installed and uninstalled a plugin using:
    inkypi plugin install mini_weather <repo_url>
    inkypi plugin uninstall mini_weather
2026-03-29_22-38

Copilot AI review requested due to automatic review settings March 30, 2026 01:39
Copy link
Copy Markdown

Copilot AI left a comment

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 aligns the inkypi plugin CLI help text and section header comments with the already-supported uninstall command, resolving the documented mismatch between “remove” and “uninstall”.

Changes:

  • Updated CLI usage output to show inkypi plugin uninstall <plugin_id> instead of remove.
  • Renamed the uninstall section header comment from # REMOVE to # UNINSTALL.

💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.

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