--help --verbose shows stack trace

[mrackwitz]

If you call a command with --verbose and --help in combination, CLAide shows a stack trace at the end of the printed help text.

$ ruby examples/make.rb --help --verbose | sed -e "s|$(pwd)|INSTALL_PATH|g"
Usage:

    $ make COMMAND

      Make delicious beverages from the comfort of yourterminal.

Commands:

    + coffee                   Drink brewed from roasted coffee beans
    + tea                      Drink based on cured leaves

Options:

    --no-milk                  Don’t add milk to the beverage
    --sweetner=[sugar|honey]   Use one of the available sweetners
    --completion-script        Print the auto-completion script
    --version                  Show the version of the tool
    --verbose                  Show more debugging information
    --no-ansi                  Show output without ANSI codes
    --help                     Show help banner of specified command

REPO/lib/claide/command.rb:338:in `help!'
REPO/lib/claide/command.rb:456:in `help!'
REPO/lib/claide/command.rb:424:in `validate!'
examples/make.rb:34:in `validate!'
REPO/lib/claide/command.rb:270:in `run'
examples/make.rb:143:in `<main>'

[segiddins]

Actually, I'm not sure that this is a bug. --verbose prints a stack trace for any uncaught exception. @irrationalfab, can I close this?

[alloy]

Printing the backtrace when the --verbose option is given is by design. While it makes less sense in the case of --help, I’d prefer that logic to stay simple and consistent.

[fabiopelosin]

@alloy why is the help an exception?

[alloy]

@irrationalfab I assume you mean the help! method? It’s so you can find the location in the source where from the apparent inability to do something is raised, making it easier to debug and fix/enhance.

[fabiopelosin]

I think that it should not be an exception if invoked by the --help flag.

[alloy]

Ok, that’s fair, so it should just print and exit from here.

@mrackwitz Are you willing to look into whipping up a patch for this?

[mrackwitz]

@alloy: Done. See #37.