Prhub

#6249 [ci] chore: remove fastmcp deps for sglang ci

原始 PR 作者 wuxibin89 合并时间 2026-05-06 13:55 文件变更 1 提交数 1 评论 1 代码增减 +2 / -2

执行摘要

移除 sglang CI 中 fastmcp 依赖

PR body 说明 'As title',即移除 fastmcp 依赖以清理 CI 配置。fastmcp 原先可能用于 sglang 测试,现已不再需要。

该 PR 简单明确,可以直接合并。无需深度审查。

讨论亮点

无 review 讨论。仅有一条 Gemini 自动评论表示无法生成 review。

实现拆解

在 sgl.yml 的两个 job(测试 sglang ServerAdapter 和 checkpoint engine)的 'Install the current repository' 步骤中,将 pip3 install hf_transfer fastmcp pytest-asyncio 改为 pip3 install hf_transfer pytest-asyncio,即删除 fastmcp 包名。改动共 2 处,各为 1 行删除和 1 行添加。

文件 模块 状态 重要度
.github/workflows/sgl.yml CI modified 2.93

关键源码片段

.github/workflows/sgl.yml infrastructure

sglang CI 工作流配置文件,移除了 fastmcp 依赖安装行。

# 在 Install the current repository 步骤中删除了 fastmcp
# 原行 : pip3 install hf_transfer fastmcp pytest-asyncio
# 新行 : pip3 install hf_transfer pytest-asyncio
steps:
  - name: Install the current repository
    run: |
      pip3 install cupy-cuda12x==13.6.0 pytest-asyncio
      pip3 install hf_transfer pytest-asyncio
      pip3 install -r requirements-test.txt
      pip3 install --no-deps -e .

评论区精华

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

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

风险与影响

低风险。若后续 sglang 测试仍然依赖 fastmcp,则 CI 会因 ImportError 失败,但 PR 提交者应已确认无依赖。回退简单,只需恢复该行。

影响范围仅限于 sglang CI 流程,减少了 fastmcp 包的安装时间(约数秒)。对用户功能和系统无影响。

无测试覆盖 依赖移除可能遗漏

关联 Issue

未识别关联 Issue

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

完整报告

参与讨论