Exiting from pre_run hooks

[devnote-dev]

For pre_run hook methods, there is the option to stop command execution to the main run method if the value returned is false. However, there are no checks in place to determine the actual exit code for this, the Executor simply returns which results in a 0 exit code:

cling/src/cling/executor.cr

Lines 52 to 54 in 279db9a

	begin
	res = resolved_command.pre_run executed.parsed_arguments, executed.parsed_options
	return unless res.nil? || res

This is obviously not ideal and even misleading for people designing applications that rely on the exit code for debugging or informational purposes. There are already ways to get around this, for example, raising a specific exception that gets funneled to the on_error hook method which then terminates the program with the appropriate exit code. But I think there should be a better way of doing this, even if it's just a shorthand method built into the Command class.

[devnote-dev]

A workaround has been implemented in 86caeef and 40e3fe2 that makes use of exception funneling as previously mentioned, meaning that the on_error hook method has been changed to accommodate for it.

While this solution works, I'm contemplating the need for such verbosity/control: if the pre_run hook method were bound to the same rules as the run and post_run hook methods—that is, not having special functionality based on the return type—then it eliminates the need for all of this and thus resolves the initial issue. 🤔