Skip to content

ci(MOC-64): no-ai-coauthor 去掉 branches:[main] 过滤(修 stacked-PR retarget 漏跑)#308

Merged
Cmochance merged 1 commit into
mainfrom
fix/moc-64-no-ai-coauthor-stacked
May 29, 2026
Merged

ci(MOC-64): no-ai-coauthor 去掉 branches:[main] 过滤(修 stacked-PR retarget 漏跑)#308
Cmochance merged 1 commit into
mainfrom
fix/moc-64-no-ai-coauthor-stacked

Conversation

@Cmochance
Copy link
Copy Markdown
Owner

@Cmochance Cmochance commented May 29, 2026

Refs MOC-64。

合 MOC-62(#307)时亲历:no-ai-coauthor.ymlon: pull_request: branches: [main],只对 base=main 的 PR 跑。stacked PR(base=feature 分支)在底 PR 合并 + 删分支后自动 retarget 到 main 时:① retarget 瞬间 base 未稳定到 main → 被 branches:[main] 过滤掉;② base retarget 本身不是触发事件类型。结果该 required check 缺失 → PR BLOCKED,只能手动 close+reopen 补跑。

改动(1 处):去掉 pull_request 的 branches: [main] 过滤,对所有 PR 跑(跟 ci.yml 一致 —— 它本就没 base 过滤)。保留 push: branches:[main] + workflow_dispatch。job 内按 PR_BASE_SHA..PR_HEAD_SHA 扫 commit,对任意 base 都正确,去掉 base 过滤无副作用。

本 PR 自身(base=main)即验证:no-ai-coauthor 对 base=main 仍正常跑。不自动 merge,等你确认。

🤖 Generated with Claude Code


Open in Devin Review

…t 漏跑

stacked PR(base=feature 分支)在底 PR 合并后自动 retarget 到 main 时,no-ai-coauthor
因 `branches:[main]` 过滤 + retarget 非触发事件类型而漏跑 → 该 required check 缺失 →
PR BLOCKED,只能手动 close+reopen 补跑(合 MOC-62 #307 时亲历)。去掉 base 过滤、对所有
PR 跑(跟 ci.yml 一致);job 内按 PR_BASE_SHA..PR_HEAD_SHA 扫 commit,任意 base 都正确。

Refs MOC-64
Copy link
Copy Markdown

@devin-ai-integration devin-ai-integration Bot left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

✅ Devin Review: No Issues Found

Devin Review analyzed this PR and found no potential bugs to report.

View in Devin Review to see 2 additional findings.

Open in Devin Review

@Cmochance Cmochance merged commit 16a4417 into main May 29, 2026
6 checks passed
@Cmochance Cmochance deleted the fix/moc-64-no-ai-coauthor-stacked branch May 29, 2026 14:02
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant