执行摘要
该 PR 将 26 个未被引用的测试/bench/CLI 文件从 python/sglang/test/ 移动到 test/manual/,并删除两个空的 __init__.py。属于代码库清理,无逻辑变更,不影响任何功能。
功能与动机
这些文件原本位于 sglang.test 安装包中,但并未接入 CI,且仓库内没有其他代码引用它们。PR 作者(hnyls2002)将其移出安装包,置于 test/manual/ 目录下,遵循已有惯例,减少包体积,保持目录结构清晰。
实现拆解
- 识别死文件:通过全库搜索确认 26 个文件在
test/、benchmark/、docs/、.github/、scripts/、Makefile 中均无引用。
- 移动文件:将文件从
python/sglang/test/ 下的子目录移动到 test/manual/ 对应的子目录,例如 python/sglang/test/ascend/disaggregation_utils.py → test/manual/ascend/disaggregation_utils.py。
- 删除空
__init__.py:python/sglang/test/attention/__init__.py 和 python/sglang/test/longbench_v2/__init__.py 因内容为空且目录下无其他文件而被删除。
无代码变更,所有移动文件内容保持不变。
评论区精华
无 review 评论。
风险与影响
- 风险:极低。文件无引用,内容不变,仅路径变化。若外部用户直接引用原路径可能失效,但仓库内已验证无引用。
- 影响:安装包体积略微减小;开发者需注意死测试已移至
test/manual/;CI 不受影响。
关联脉络
该 PR 与近期测试目录整理(PR #24725)一脉相承,将未接入 CI 的测试进一步分离到 test/manual/,使 test/registered/ 专注于 CI 测试。
参与讨论