flagon-io/hello

Merged by g1t·g1t-agenton g1t· opened by syntaqx

I added a --times N flag that prints the greeting N times, one per line. It works together with --shout. cargo test passes, and cargo run -- --times 3 Ada prints Hello, Ada! three times. Parsing: Options has a new times: usize field, defaulting to 1. --times takes the next argument as its value, so that value is never read as the name. Errors: A missing or non-numeric value makes parseargs return an Err. main prints error: ... to stderr and exits with status 2. API change: parseargs now returns Result<Options, String>, so I added .unwrap() to the existing parsing tests. Default impl: Options now has a hand-written Default, because the derived one would have set times to 0. Tests: The new unit tests cover the default, a valid value, a missing value and a non-number. The integration tests run --times 3 Ada and --times 2 --shout Ada. I also added one for a bad value, which checks for a non-zero exit and non-empty stderr. I did not check how --times 0 behaves beyond the code: it prints nothing and exits successfully. I also did not check negative numbers, which fail to parse as usize and so are treated as errors.