Skip to content

fix(cli): Avoid panic when config fails to load - #1973

Merged
gtema merged 1 commit into
mainfrom
fix2
Sep 3, 2026
Merged

fix(cli): Avoid panic when config fails to load#1973
gtema merged 1 commit into
mainfrom
fix2

Conversation

@gtema

@gtema gtema commented Sep 3, 2026

Copy link
Copy Markdown
Owner

Config::new().expect(...) as a clap default_value_t ran on every CLI
invocation; an unreadable/malformed default config, or an unavailable
config dir, panicked before the actual command ran. Fall back to
Config::default() instead.

Signed-off-by: Artem Goncharov artem.goncharov@gmail.com

Config::new().expect(...) as a clap default_value_t ran on every CLI
invocation; an unreadable/malformed default config, or an unavailable
config dir, panicked before the actual command ran. Fall back to
Config::default() instead.

Signed-off-by: Artem Goncharov <artem.goncharov@gmail.com>
@gtema
gtema merged commit 5d1d43e into main Sep 3, 2026
20 checks passed
@gtema-release-plz gtema-release-plz Bot mentioned this pull request Sep 3, 2026
@gtema
gtema deleted the fix2 branch September 3, 2026 15:42
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.

1 participant