Prhub

#42160 [Docs] Fix broken local links

原始 PR 作者 chfeng-cs 合并时间 2026-05-10 16:15 文件变更 2 提交数 4 评论 7 代码增减 +2 / -2

执行摘要

修复两个文档中的错误相对链接

PR 描述指出,benchmarking CLI 页面的 timeline HTML 资产路径解析到了 docs/assets 外部,而 EPD 集成测试 README 的分解编码器文档路径中的 ../ 段数不足,在 tests/docs 下解析而非仓库根目录的 docs 目录。

该 PR 可直接合并,无需深入精读。值得关注的是作者对 CI 失败的正确判断和分离修复策略,可作为文档类 PR 的参考实践。

讨论亮点

无实质性讨论。合并者 noooop 简单表示感谢,并指出由于 mypy CI 失败 (与 #42197 有关) 自动合并暂停;提交者 chfeng-cs 确认 mypy 失败与 PR 无关,将另开 PR 修复测试文件。

实现拆解

该 PR 对两个 Markdown 文件各做了一行路径调整:

  1. docs/benchmarking/cli.md:将 iframe 的 src 属性从 ../../assets/contributing/vllm_bench_serve_timeline.html 改为 ../assets/contributing/vllm_bench_serve_timeline.html,减少一层父目录,使其能在 docs/benchmarking/ 下正确定位到 docs/assets/
  2. tests/v1/ec_connector/integration/README.md:将指向 docs/features/disagg_encoder.md 的相对路径从 ../../../docs/features/disagg_encoder.md 改为 ../../../../docs/features/disagg_encoder.md,增加一层父目录,使其能从 tests/v1/ec_connector/integration/ 正确回溯到仓库根目录的 docs/

所有修改均为纯文档变更,不涉及源代码或测试逻辑。

文件 模块 状态 重要度
docs/benchmarking/cli.md 文档 modified 1.72
tests/v1/ec_connector/integration/README.md 文档 modified 1.72

分析完成后,这里会展示 LLM 生成的相对完整源码片段和详细注释。

评论区精华

CI mypy 检查失败与 PR 无关 other

提交者 chfeng-cs 指出 mypy 失败是由于测试文件 `tests/compile/correctness_e2e/test_sequence_parallel.py` 中的 `enable_prompt_embeds` 参数与当前 `_compare_sp` 签名不同步,与文档更改无关。

结论:提交者将另开 PR 修复测试文件,合并者 noooop 表示需等待 #42197 使 CI 全部通过后才能自动合并。 · 已解决

风险与影响

风险极低。仅两处 Markdown 链接修改,不会影响任何功能、性能或安全性。唯一潜在风险是路径仍然错误,但可通过手动验证或文档预览确认。

仅影响文档阅读体验,修复了两个断链。对用户使用无功能影响。

无风险

关联 Issue

未识别关联 Issue

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

完整报告

参与讨论