Skip to content

Exit gracefully when cli args parsing fails#175

Merged
Toqozz merged 1 commit into
Toqozz:masterfrom
leana8959:graceful-exit
Apr 24, 2026
Merged

Exit gracefully when cli args parsing fails#175
Toqozz merged 1 commit into
Toqozz:masterfrom
leana8959:graceful-exit

Conversation

@leana8959

Copy link
Copy Markdown
Contributor

Might be a nitpick but I think it's nice to not panic when it comes to cli option parsing which can be user errors.

Previously:

thread 'main' panicked at src/cli.rs:258:19:
Argument to option 'dnd' missing
note: run with `RUST_BACKTRACE=1` environment variable to display a backtrace

Current:

User error: Argument to option 'dnd' missing

@leana8959 leana8959 marked this pull request as ready for review April 24, 2026 12:46
@Toqozz

Toqozz commented Apr 24, 2026

Copy link
Copy Markdown
Owner

I wholeheartedly agree. I'm not sure what I was thinking when I wrote that code.

Comment thread src/cli.rs Outdated
@Toqozz Toqozz merged commit 95edd86 into Toqozz:master Apr 24, 2026
3 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants