执行摘要
移除 sglang CI 中 fastmcp 依赖
PR body 说明 'As title',即移除 fastmcp 依赖以清理 CI 配置。fastmcp 原先可能用于 sglang 测试,现已不再需要。
该 PR 简单明确,可以直接合并。无需深度审查。
无 review 讨论。仅有一条 Gemini 自动评论表示无法生成 review。
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 链接,后续同步到相关引用后会出现在这里。
参与讨论