# PR #1940 完整报告

- 仓库：`THUDM/slime`
- 标题：run cpu test on main branch
- 合并时间：2026-05-25 15:01
- 原文链接：http://prhub.com.cn/THUDM/slime/pull/1940

---

# 执行摘要

- 一句话：在 main 分支 push 时自动运行 CPU 单元测试
- 推荐动作：该 PR 是基础设施优化，值得关注 CI 配置最佳实践。建议合并。

# 功能与动机

在 main 分支合并后自动检测回归，确保主线稳定性。

# 实现拆解

1. 在 `.github/workflows/pr-test.yml.j2` 和 `.github/workflows/pr-test.yml` 中，将 `push` 事件重新启用，限制触发分支为 `main`。
2. 在 CPU 单元测试任务的触发条件中增加 `github.event_name == 'push'`，使 push 到 main 时执行该任务。
3. 添加注释说明：push 事件仅触发 CPU 测试（低成本），GPU 任务仍依赖标签以避免消耗自托管资源。

关键文件：
- `.github/workflows/pr-test.yml`（模块 CI；类别 infra；类型 infrastructure）: 实际 CI 配置，修改了 push 事件触发和 cpu-unittest 的条件。
- `.github/workflows/pr-test.yml.j2`（模块 CI；类别 infra；类型 infrastructure）: CI 模板配置，与 pr-test.yml 同步修改，是配置的源头。

关键符号：未识别


# 评论区精华

无 review 讨论。

- 暂无高价值评论线程

# 风险与影响

- 风险：变更仅涉及 CI 触发条件，不涉及代码逻辑，风险极低。主要风险是 push 到 main 时若 CPU 测试发现失败，可能阻塞后续合并，但这是期望行为。
- 影响：每次 push 到 main 分支都会触发 CPU 单元测试，增加少量 CI 运行次数，但使用 GitHub 托管 runner，成本可控。GPU 任务不受影响。
- 风险标记：低风险，仅 CI 变更

# 关联脉络

- PR #1939 add more cpu ci: 同样涉及 CPU 单元测试的 CI 配置改进，本 PR 扩展了触发场景。