fix(ci): read only GitHub Actions' ci-status runs in the release rebuild decision
维护者通常 1 天内回复
已经有一个关联 PR 被合并了。
- #9653 来自 @kyle-sexton —— 已合并
评估
调研方向
阅读 .github/workflows/release-publish-plugins.yml 第 116 行附近以及 .github/scripts/release-publish-plugins/decide-rebuild.sh，找到两处读取 check run 的代码；将它们与 .github/scripts/pr-merge/gate-check.mjs 第 56 行附近的代码进行比较。添加测试，证明来自其他 app、时间更晚且成功的 ci-status 不会决定是否重建发布，并确保开始时间相同或无法读取时仍以失败关闭。
由索引模型根据 Issue 内容生成。
描述
Refs: #9610
Problem
.github/workflows/release-publish-plugins.yml:116 and .github/scripts/release-publish-plugins/decide-rebuild.sh read ci-status check runs without an app_id filter and pick the newest by started_at. Another GitHub App can post a check run named ci-status with a later start time and win that pick. The ruleset (ci-status required from the GitHub Actions app) and the merge queue still protect merging, but the release path decides on its own read.
Fix
Add app_id=15368 to those reads, as .github/scripts/pr-merge/gate-check.mjs:56 and pr-merge's Dependabot gate do, and keep ties and unreadable start times failing closed.
Acceptance criteria
- Both reads filter on the GitHub Actions app id
- A test shows another app's later
ci-statussuccess does not decide
- 主要语言
- Shell
- 星标
- 22
- 派生
- 2
- 平均合并
- 3 小时 19 分钟
- 30 天内合并 PR
- 955
环境准备
- 没有 Dockerfile 或 Docker Compose 文件
- 没有 Pull Request 模板
- 阅读贡献指南
从这里开始
- 先读完整个 Issue，再读项目的贡献指南。
- 在 Issue 下留言说明你要接手 —— 这能避免两个人做同样的事。
- Fork 仓库，在一个分支上完成修改。
- 提交 Pull Request，并在描述里引用这个 Issue 编号。
melodic-software/claude-code-plugins 的其他 Issue
-
priority: critical status: needs-decision work-class: scoped
难度 2/5 1-3 小时 新手友好度 72/100
melodic-software/claude-code-plugins#10492 · 4 条评论 ·
维护者通常 1 天内回复
-
priority: medium status: ready work-class: scoped
难度 2/5 1-3 小时 新手友好度 76/100
melodic-software/claude-code-plugins#10227 · 1 条评论 ·
维护者通常 1 天内回复
-
fix(evals): give the methodology rewrite answer a template whose target slot needs a stated basis未关闭priority: medium status: ready work-class: scoped
难度 2/5 1-3 小时 新手友好度 72/100
melodic-software/claude-code-plugins#10217 · 1 条评论 ·
维护者通常 1 天内回复
-
priority: medium status: ready work-class: scoped
难度 2/5 1-3 小时 新手友好度 68/100
melodic-software/claude-code-plugins#10128 · 1 条评论 ·
维护者通常 1 天内回复
-
pr-pipeline: clear a feedback-not-landed escalation once the head moves and no lane thread is open未关闭priority: medium status: needs-decision work-class: scoped
难度 2/5 1-3 小时 新手友好度 83/100
melodic-software/claude-code-plugins#10074 · 4 条评论 ·
维护者通常 1 天内回复
查看 melodic-software/claude-code-plugins 的全部 Issue
相似的 Issue
-
难度 1/5 1 小时以内 新手友好度 72/100
-
source:reader type:content-error
难度 1/5 1 小时以内 新手友好度 88/100
stephrobert/blog-roadmap#136 ·
-
add `git-lfs`未关闭
难度 2/5 1-3 小时 新手友好度 62/100
-
难度 2/5 1-3 小时 新手友好度 68/100
ophub/amlogic-s9xxx-armbian#3752 ·
维护者通常 1 天内回复
-
prio:low type:chore
难度 1/5 1 小时以内 新手友好度 82/100
davidsneighbour/dotfiles#602 ·
维护者通常 3 天内回复