Skip to content

[Feature] PR CI 状态闭环:失败提醒修复,ready 后提醒合入 #121

Description

@zengchuang-hw

Summary

补齐 PR CI 状态闭环:required CI 失败时提醒贡献者修复;CI 恢复且 PR 满足 ready 条件后,提醒对应 committer/maintainer 合入。

Parent tracking issue: #116

Proposed behavior

CI failed

  • required Buildkite/GitHub checks 变红时,在 PR 上 @ author。
  • 如果最后一次非 merge commit 的 author/pusher 与 PR author 不同,一并提醒。
  • 评论包含失败 job、head SHA、CI 链接和明确的修复请求。
  • 跳过 bot、GitHub Actions 和 Dependabot。

CI green + ready

当以下条件同时成立时,提醒有合入权限的 committer/maintainer:

  • PR 非 draft;
  • required checks 全绿;
  • mergeable;
  • 满足仓库要求的 approval/ready label/其他 merge gate。

提醒中包含 PR、head SHA、ready 判定依据,并 @ 路由到的 committer/maintainer 请求合入。

Idempotency

  • 使用 (repo, PR, head SHA, lifecycle state) 去重。
  • 同一 SHA 的 CI 重跑不重复提醒。
  • head 更新或状态从 ci_failed 有效迁移到 ready_to_merge 后,允许产生对应的新提醒。

Acceptance criteria

  • CI 失败评论包含失败 job、head SHA、链接和正确的贡献者 mention。
  • CI 全绿且满足 ready 条件后,正确提醒 committer/maintainer 合入。
  • 同一 head/state 不重复评论,服务重启后仍保持幂等。
  • CI 绿但 draft、不可合并或 approval/label 不足时不发送合入提醒。
  • bot/Dependabot 不会被 mention。
  • 不自动修代码、不自动 merge,所有写入继续受 ALLOW_POST/posting gate 控制。
  • 首期仅对 vllm-project/vllm-omni 开启。

Related

Activity

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Metadata

Metadata

Assignees

No one assigned

    Labels

    No labels
    No labels

    Type

    No type

    Projects

    No projects

      Milestone

      No milestone

      Relationships

      None yet

      Development

      No branches or pull requests

      Issue actions