Skip to content

`clive configure key remove` command and associated improvements

Mateusz Żebrak requested to merge mzebrak/remove-key into develop

Closes: #88 (closed) and #86 (closed)

This MR:

  • Add clive configure key remove command
  • Create a ContextualCLICommand abstraction
  • Improves commands that might optionally require beekeeper (it won't be launched until needed)
  • Improve ExternalCLICommand abstract class by providing a common validate method (options could be validated before taking any further actions)
  • Improve ExternalCLICommand abstract class by providing a common _configure method (Subclasses could dynamically change their run configuration based e.g. on options given like e.g. don't use the beekeeper when other param was not given)
  • It is now impossible to remove a key from the beekeeper from the TUI (only CLI with explicit switch)
Edited by Mateusz Żebrak

Merge request reports

Loading