Prhub

#24179 Revert "ci: add rebase-required mode to check-maintenance action"

原始 PR 作者 Kangyan-Zhou 合并时间 2026-05-01 07:11 文件变更 2 提交数 1 评论 1 代码增减 +26 / -101

执行摘要

回退 rebase-required CI 检测模式

该 PR 的出发点是对 #23109 的完全回退,具体回退原因未在 PR 描述中说明。根据提交信息,回退可能是为了消除 rebase-required 模式带来的额外复杂性,或由于该模式在实际使用中出现了问题。原始功能允许通过 issue body 中的 MIN_BASE_SHA 指令要求所有 PR 必须包含某个基准提交才能运行 CI,该回退移除了这一能力。

该 PR 是纯粹的回退操作,变更简单直接。建议关注回退后的 CI 稳定性,以及是否有替代方案来满足原始需求(要求 PR rebase)。如果不需要该功能,可以放心合入。

讨论亮点

本 PR 无 review 评论和讨论。

实现拆解

回退的实现过程分为两步:

  1. .github/actions/check-maintenance/action.yml 中删除所有与 rebase-required 模式相关的代码,包括从 issue body 解析 MIN_BASE_SHA、通过 GitHub Compare API 检查 PR 是否包含指定 SHA、以及相应的警告和退出逻辑;同时简化描述、去除不需要的输入变量(如 PR_HEAD_SHA)。
  2. .github/MAINTAINER.md 中移除整节 "Rebase-Required Mode" 文档,恢复原始维护说明。

此回退不涉及测试、配置或部署的配套改动。

文件 模块 状态 重要度
.github/actions/check-maintenance/action.yml CI 动作 modified 4.79
.github/MAINTAINER.md 维护文档 modified 2.43

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

评论区精华

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

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

风险与影响

主要风险包括:1)如果其他团队或工作流已依赖 rebase-required 模式,回退将使其失效;2)变更属于 CI 基础设施,一旦部署将直接影响所有 PR 的 CI 流程;3)回退动作没有经过公开讨论,可能忽略了对该功能有需求的场景。但回退本身是恢复历史行为,从版本回退角度看风险可控。

影响范围:仅限 CI 维护流程和仓库维护文档。对用户模型推理无影响。对开发者的影响:维护者无法再通过 issue 体指令要求 PR 必须 rebase 到特定提交。影响程度较低,因为该功能在 #23109 合并后尚未广泛使用。

回退移除功能 CI 流程变更 缺乏讨论

关联 Issue

#23109 ci: add rebase-required mode to check-maintenance action

完整报告

参与讨论