Prhub

#25927 fix: rustfmt service_discovery.rs warn! line length

原始 PR 作者 ch-wan 合并时间 2026-05-21 10:25 文件变更 1 提交数 1 评论 0 代码增减 +3 / -1

执行摘要

修复 rustfmt 行长度警告

PR body 明确指出:"Reformats a warn! macro call in sgl-model-gateway/src/service_discovery.rs to satisfy rustfmt line-length rules. Introduced by #25294." 目的是修复代码风格问题,确保符合 rustfmt 标准。

可直接合并。该 PR 是纯代码风格修正,无需特别关注设计决策或深度审查。

讨论亮点

该 PR 无 review 评论讨论。审核机器人 gemini-code-assist[bot] 仅表示没有反馈。

实现拆解

  1. 定位问题代码:在 sgl-model-gateway/src/service_discovery.rs 文件的 PodInfo::should_include 方法中,第 118 行的 warn! 宏调用字符串过长,触发 rustfmt 行长度警告。
  2. 格式化修复:将 warn! 宏的参数拆分为多行,使其符合 rustfmt 默认的行长度限制(通常为 100 或 120 字符)。
  3. 无逻辑变更:仅调整代码格式,未修改任何功能逻辑。
文件 模块 状态 重要度
sgl-model-gateway/src/service_discovery.rs 模型网关 modified 4.29

关键源码片段

sgl-model-gateway/src/service_discovery.rs style

唯一变更文件,修正了 `warn!` 宏的行长度问题,确保通过 rustfmt 检查。

// 修复前:单行过长
warn!("PD mode enabled but both prefill_selector and decode_selector are empty");// 修复后:拆分为多行,满足 rustfmt 行长度限制
warn!(
    "PD mode enabled but both prefill_selector and decode_selector are empty"
);

评论区精华

没有提炼出高价值讨论线程

当前评论区没有形成足够清晰的争议点或结论,后续有更多讨论时会体现在这里。

风险与影响

极低风险:仅格式化变更,不涉及逻辑修改。回归风险几乎为零,无性能、安全或兼容性影响。

  • 用户:无影响。
  • 系统:无影响。
  • 团队:有助于保持代码风格一致性,避免 CI 中 rustfmt 检查失败。
  • 影响范围:单一文件,单行代码。

关联 Issue

未识别关联 Issue

当前没有检测到明确关联的 Issue 链接,后续同步到相关引用后会出现在这里。

完整报告

参与讨论