Prhub

#21753 [CI] Fix rerun-test suite detection to skip commented registrations

原始 PR 作者 ispobock 合并时间 2026-03-31 18:00 文件变更 1 提交数 1 评论 3 代码增减 +3 / -1

执行摘要

修复 CI 中 rerun-test 命令检测测试套件时忽略注释行的正则表达式。

根据 PR body 中的动机部分,引用自 PR #21751 的评论(链接 https://github.com/sgl-project/sglang/pull/21751#issuecomment-4161252298),需要修复正则表达式以跳过注释掉的 register_cuda_ci 行,避免在运行 CI 测试时误检测。

这是一个简单的 CI 工具修复,对大多数工程师无需精读。如果有兴趣了解 CI 测试套件检测逻辑,可以查看 detect_cuda_suite 函数的实现。

讨论亮点

Review 中没有实质性讨论,仅有一个 gemini-code-assist[bot] 的自动评论表示没有反馈。因此,无争议点或设计权衡。

实现拆解

修改了 scripts/ci/utils/slash_command_handler.py 中的 detect_cuda_suite 函数。关键改动点:正则表达式从 r'register_cuda_ci([^)]suite\s=\s"\'["\']' 更新为 r'^[^#\n]register_cuda_ci([^)]suite\s=\s"\'["\']',其中 ^[^#\n] 确保匹配行不以 # 开头;添加 re.MULTILINE 标志,使 ^ 匹配每行的开头。

文件 模块 状态 重要度
scripts/ci/utils/slash_command_handler.py CI 工具 modified 3.0

关键符号

detect_cuda_suite

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

评论区精华

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

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

风险与影响

风险较低,正则表达式变更简单且针对性强。可能的风险是如果正则表达式仍不完善,可能漏掉某些非注释行或错误匹配其他模式,但基于变更内容,风险可控。

直接影响 CI 中 /rerun-test 命令的行为,确保只运行有效的测试套件注册,避免因注释行导致的误测试。对用户无直接影响,仅影响开发团队的 CI 流程效率。

正则表达式变更 低风险变更

关联 Issue

未识别关联 Issue

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

完整报告

参与讨论