dgf1988/codewhale
1
0
Fork 0
Code Issues Pull Requests Actions 1 Packages Projects Releases Wiki Activity

style(config): mark unreachable DeepSeek match arms (review #1740)

Browse Source 
Address gemini-code-assist review on PR #1740 (MEDIUM, clarity): the
non-DeepSeek else-branch match listed ApiProvider::Deepseek /
DeepseekCN arms that are logically unreachable (handled by the if
branch above). Collapse to a single
'Deepseek | DeepseekCN => unreachable!(...)' arm so the intent is
explicit for future maintainers. No behavior change.

Refs #1714.
This commit is contained in:
Zhongyue Lin
2026-05-18 01:50:15 +08:00
committed by Hunter Bown
parent b156d7da33
commit 3adc05d627
1 changed files with 1 additions and 2 deletions
Download Patch File Download Diff File Expand all files Collapse all files
crates/tui/src/config.rs
+1 -2
View File
@@ -2392,8 +2392,7 @@ fn apply_env_overrides(config: &mut Config) {
.providers
.get_or_insert_with(ProvidersConfig::default);
let entry = match provider {
ApiProvider::Deepseek => &mut providers.deepseek,
ApiProvider::DeepseekCN => &mut providers.deepseek_cn,
ApiProvider::Deepseek | ApiProvider::DeepseekCN => unreachable!("DeepSeek providers are handled in the if branch above (issue #1714)"),
ApiProvider::NvidiaNim => &mut providers.nvidia_nim,
ApiProvider::Openai => &mut providers.openai,
ApiProvider::Atlascloud => &mut providers.atlascloud,