skraus-dev / cherryrgb-rs

Cherry RGB Keyboard - Multi platform tool to set RGB LEDs for Cherry keyboards
MIT License
41 stars 3 forks source link

CLI animation command wrongly accepts "custom" as parameter #52

Closed felfert closed 1 year ago

felfert commented 1 year ago

Describe the bug In the help output, custom is listed as possible value for the animation mode and when invoking the cli in this way, no error is produced but the command obviously does fail. custom is handled by the subcommands custom-colors and color-profile-file and should neither be listed in the possible values, nor be accepted by the animation subcommand.

Version I believe, this exists since the beginning.

To Reproduce Steps to reproduce the behavior:

  1. Call the application with following argument animation custom fast ffffff
  2. No error and all LEDs turn off.

Expected behavior

Additional context N/A