feat: Add CommandInput

[aawsome]

Add the new type CommandInput. This type supports serde and clap (using FromStr) and is supposed to be used where users need to specify commands which are to be executed.

There are 3 ways to define commands (here TOML is used):

• as one string: command = "echo test"
• as command and args as string array: command = {command = "echo", args = ["test1", "test2"] }
• additionally specify error treatment: command = {command = "echo", args = ["test1", "test2"], on_failure = "ignore" }

In the first example the full command and the args are split using shell_words which requires special escaping or quoting when spaces are contained. In the other examples every string is taken literally as given (but TOML escapes may apply when parsing TOML).

This PR only adds the new type, using it on existing commands will be a breaking change.

[codecov]

Codecov Report

Attention: Patch coverage is 62.79070% with 16 lines in your changes missing coverage. Please review.

Project coverage is 47.5%. Comparing base (8ef75e6) to head (82d2302).
Report is 1 commits behind head on main.

Files with missing lines	Patch %	Lines
crates/core/src/repository/command_input.rs	62.7%	16 Missing ⚠️

Additional details and impacted files

Files with missing lines	Coverage Δ
crates/core/src/error.rs	57.1% <ø> (ø)
crates/core/src/repository.rs	41.2% <ø> (-0.3%)	⬇️
crates/core/src/repository/command_input.rs	62.7% <62.7%> (ø)

... and 15 files with indirect coverage changes

[aawsome]

@nardoor thanks a lot for the review and the very valuable suggestions!
I changed basically all as you suggested and think this is ready for a re-review.

The only question I have is about the naming of on_failure - maybe on_error is a better name, what do you think?

[nardoor]

@nardoor thanks a lot for the review and the very valuable suggestions!
I changed basically all as you suggested and think this is ready for a re-review.

The only question I have is about the naming of on_failure - maybe on_error is a better name, what do you think?

I feel like on_failure is fine, as it has a broader meaning.

I'm glad you found my comments useful. I'll look at your changes tomorrow!

[nardoor]

lgtm!