Skip to content

ci: 恢复 #174 PR 候选基准修复供重新审阅 - #182

Closed
3316891527 wants to merge 1 commit into
fix/revert-pr-170-176from
ci/reopen-pr-174-candidate-base
Closed

3316891527 wants to merge 1 commit into
fix/revert-pr-170-176from
ci/reopen-pr-174-candidate-base

Conversation

@3316891527

@3316891527 3316891527 commented Nov 9, 2026 •

Copy link
Copy Markdown
Owner

变更说明 / Description

PR 事件保存的 base 落后于实际合并候选时,以固定候选的第一父提交作为真实比较基准,避免正确候选被误报为污染。事件 base 必须是第一父祖先,第二父仍精确匹配事件 head,且候选恰有两个父提交;差异比较使用 candidate^1..candidate。

背景与动机 / Context and motivation

维护者要求撤销 #170–#176 并恢复 open 审阅,范围包含原 #174 的 CI 修复。原 PR 已合并无法重开,本 PR 保留其补丁供重新审阅;未经维护者明确允许不得合并或启用自动合并。

改动范围 / Changes

完整恢复 ci/script/pr_check.py、Python 门禁回归及文档,共 5 文件。工作流 YAML 不变。

兼容性与风险 / Compatibility and risks

调整的是候选身份校验和真实基准选择。撤销后严格匹配旧事件 base 的行为也会恢复,可能再次产生误报。本 PR 暂以撤销 PR #177 的分支为目标展示独立差异;待 #177 获准进入 dev 后应改为 dev 并重新检查,不能合入临时基准分支。

关联 Issue / Related issue

原 #174;撤销与恢复审阅要求见 #177。

验证方式 / Verification

初次重建时已逐项核对原完整合并补丁。本轮在清理历史空白并补齐 CI 的共同回滚基准上重放该项,保持独立功能差异。当前 PR 启用 ci:full-android,要求固定合并候选实际运行 assembleDebug 与 :app:testDebugUnitTest,Candidate checks 必须检查两项真实成功。新的结果以本 PR Checks 为准;历史 CI 不代替本轮构建和测试。

证据 / Evidence

原实现与历史检查见 #174,回滚基准见 #177。

检查清单 / Checklist

  • 已记录静态核验、历史证据与未运行项原因。
  • 原 CI 补丁、回归和文档完整保留。
  • 待 revert: 撤销 PR #170–#176,恢复合入前代码 #177 获准合入后将目标改为 dev。
  • 本轮固定候选 assembleDebug、testDebugUnitTest 与 Candidate checks 实际成功。
  • 等待维护者明确批准合并。

本项已随 集成 PR #191 合入 dev,并经 发布同步 PR #193 同步到 main。原来源提交 eb8f4885a3f3b986eebb193dad3fe97556bf9c9d 已核对保留。

本 PR 原目标为 fix/revert-pr-170-176;由于 dev 已包含全部来源提交,GitHub 拒绝将目标改到 dev。关闭此重复待合并条目,代码与原始审阅记录均保留。

@3316891527 3316891527 added the ci:full-android Require assembleDebug and Android JVM tests for this PR candidate label Nov 9, 2026
Reapply the complete original merge diff from PR #174 on the reviewed rollback base.
Original merge: 51e8efa

Keep this work open for maintainer review. No merge or auto-merge without explicit permission.
@3316891527
3316891527 force-pushed the ci/reopen-pr-174-candidate-base branch from c05514a to eb8f488 Compare November 9, 2026 03:16
@3316891527 3316891527 closed this Nov 10, 2026
@3316891527
3316891527 deleted the ci/reopen-pr-174-candidate-base branch November 10, 2026 04:27
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

ci:full-android Require assembleDebug and Android JVM tests for this PR candidate

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant