Prhub

#24979 [Doc] Add rerun-test slash command usage

原始 PR 作者 ispobock 合并时间 2026-05-11 21:30 文件变更 1 提交数 1 评论 2 代码增减 +1 / -0

执行摘要

文档新增 /rerun-test 命令说明

当前 CI 流程中缺少对 /rerun-test 命令的文档说明,导致开发者不知道如何通过评论触发特定测试的重跑。PR body 明确指出 "Update /rerun-test usage to doc"。

建议合并,但可考虑根据 review 反馈优化 test-spec 格式的描述,明确区分标准 pytest 分隔符与本命令的实际分隔符。

讨论亮点

gemini-code-assist[bot] 指出文档中使用 "pytest-style" 标签但采用了点号(.)分隔类和方法,这与标准 pytest 使用 :: 分隔所有层级不一致,建议直接描述格式而不使用 "pytest-style" 称谓以避免混淆。该评论未得到回复。

实现拆解

  1. docs_new/docs/developer_guide/contribution_guide.mdx 文件中,于斜杠命令列表末尾新增一行关于 /rerun-test 的文档条目。
  2. 内容详细说明了命令格式:/rerun-test <test-spec> [<test-spec> ...],其中 <test-spec> 采用 pytest 风格的 file::TestClass.method 格式,且 TestClass 和方法部分可选。
  3. 补充了命令行为:将根据注册的 runner-label 分组并触发对应的重跑工作流。
  4. 提供了多个示例和权限说明。
文件 模块 状态 重要度
docs_new/docs/developer_guide/contribution_guide.mdx 开发者文档 modified 2.0

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

评论区精华

test-spec 格式描述与 pytest 标准不一致 documentation

gemini-code-assist[bot] 指出文档中“pytest-style”的描述与示例中使用点号分隔类和方法不符,因为标准 pytest 使用 `::` 分隔所有层级。建议直接描述格式,避免混淆。

结论:评论未收到回复,文档保持原样。 · unresolved

风险与影响

仅文档变更,无代码逻辑修改,不存在回归、性能或安全风险。但文档中关于 test-spec 格式的描述存在歧义,可能误导用户。

影响范围小,仅涉及贡献者文档的一个段落。有助于开发者了解如何使用 /rerun-test 命令,提升 CI 调试效率。

关联 Issue

未识别关联 Issue

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

完整报告

参与讨论