Repository navigation
ci: 恢复 #174 PR 候选基准修复供重新审阅 - #182
Closed
3316891527 wants to merge 1 commit into
Closed
3316891527 wants to merge 1 commit into
3316891527 wants to merge 1 commit into
Conversation
2 of 5 tasks
3316891527
force-pushed
the
ci/reopen-pr-174-candidate-base
branch
from
November 9, 2026 03:16
c05514a to
eb8f488
Compare
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
变更说明 / 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
本项已随 集成 PR #191 合入
dev,并经 发布同步 PR #193 同步到main。原来源提交eb8f4885a3f3b986eebb193dad3fe97556bf9c9d已核对保留。本 PR 原目标为
fix/revert-pr-170-176;由于dev已包含全部来源提交,GitHub 拒绝将目标改到dev。关闭此重复待合并条目,代码与原始审阅记录均保留。