Prhub

#45117 Mergify message not on cancelled

原始 PR 作者 hmellor 合并时间 2026-07-26 00:20 文件变更 1 提交数 5 评论 0 代码增减 +1 / -0

执行摘要

修复 Mergify 对已取消 CI 任务发送评论的问题

GitHub Actions 的并发取消机制会导致之前运行的 CI 任务被标记为 'cancelled',Mergify 的 check-failure 条件错误地将 cancelled 视为 failure 从而触发评论,造成无效通知。Issue #5172 明确指出 'Treating cancelled jobs as failures causes Mergify to spam active PRs'。

该 PR 为简单配置调整,无需仔细审查,但对了解 vLLM 的 CI 工作流和 Mergify 配置有参考价值。

讨论亮点

该 PR 无 review 评论,仅获得一个 approve,无争议。

实现拆解

.github/mergify.ymlpull_request_rules.comment-pre-commit-failure 规则的 conditions 列表里,在 check-failure=pre-commit 之后添加一条条件:-check-cancelled=pre-commit

  • 文件:.github/mergify.yml
  • 变更:仅增加一行高亮,无其他文件修改。
文件 模块 状态 重要度
.github/mergify.yml CI 配置 modified 2.4

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

评论区精华

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

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

风险与影响

几乎无风险。仅修改 CI/CD 配置规则,不影响代码功能或用户行为。

影响范围局限于使用了 Mergify 的仓库 CI 评论行为,减少不必要的 spam 评论,提升开发者体验。

关联 Issue

#5172 `check-failure` condition should not include cancelled

完整报告

参与讨论