fix(ci): signed-commit check fails pr-refine on PRs over 250 commits even when the lane made no commits
メンテナーはふだん 1 日以内に返信
まだ誰も着手していません。
評価
- 難易度
- 4/5
- 見積もり時間
- 3〜5日
- 初心者へのやさしさ
- 57/100
調査の方向性
Start with .github/actions/check-signed-commits/check-signed-commits.mjs, especially the 250-commit cap, completeness check, and escalation label request. Trace how the action identifies commits made by the lane and how it handles a lane with no commits; use the observed run on #7009 as a reference. Done means the check covers only the lane's commits, skips empty lanes, and a failed label write warns without failing the job.
索引モデルが issue の本文から書いたものです。
説明
Problem
On a pull request with more than 250 commits, pr-refine / fix-docs fails even when the lane made no commits.
.github/actions/check-signed-commits/check-signed-commits.mjs:14setsLISTED_COMMITS_CAP=250. Past the cap, line 92 setscomplete=false, soallVerifiedis false even when every listed commit is verified.- The escalation then POSTs a label to
issues/<n>/labels. That fails with 403 because the token has no issues write, and the failed POST becomes a hard failure. - The check runs even when the lane produced no commits ("The activity made no commits"), so there is nothing of the lane's to check.
Observed on #7009 (259 commits), run 38179498135: check-run summary "A commit on the pull request is not GitHub-verified."; step "Check the lane's commits are signed": "POST .../issues/7009/labels failed with status 403".
Expected
- Skip the signed-commit check when the lane made no commits.
- Check only the lane's own commits, not the PR's full history, so the 250-commit list cap does not apply.
- A failed escalation label write should be a warning, not a job failure.
- 主要言語
- Shell
- スター
- 22
- フォーク
- 2
- 平均マージ
- 8時間 25分
- マージ済み PR（30日）
- 944
環境構築
- Dockerfile・Docker Compose ファイルなし
- プルリクエストのテンプレートなし
- コントリビューションガイドを読む
はじめの一歩
- issue を最後まで読み、次にプロジェクトのコントリビューションガイドを読みます。
- 着手することを issue にコメントします — 二人が同じ作業をするのを防げます。
- リポジトリをフォークし、ブランチを切って変更します。
- issue 番号を参照したプルリクエストを送ります。
melodic-software/claude-code-plugins のほかの issue
-
good first issue priority: low
難易度 2/5 1〜3時間 初心者へのやさしさ 62/100
melodic-software/claude-code-plugins#7530 · コメント 1 件 ·
メンテナーはふだん 1 日以内に返信
-
needs-human priority: medium
難易度 2/5 1〜3時間 初心者へのやさしさ 65/100
melodic-software/claude-code-plugins#7455 · コメント 1 件 ·
メンテナーはふだん 1 日以内に返信
-
priority: medium
難易度 2/5 1〜3時間 初心者へのやさしさ 68/100
melodic-software/claude-code-plugins#7351 · コメント 2 件 ·
メンテナーはふだん 1 日以内に返信
-
agent-ready priority: high work-class: scoped
難易度 2/5 1〜3時間 初心者へのやさしさ 74/100
melodic-software/claude-code-plugins#7312 · コメント 3 件 ·
メンテナーはふだん 1 日以内に返信
-
priority: medium
難易度 2/5 1〜3時間 初心者へのやさしさ 76/100
melodic-software/claude-code-plugins#7288 · コメント 2 件 ·
メンテナーはふだん 1 日以内に返信
melodic-software/claude-code-plugins の issue をすべて見る
似ている issue
-
Add dependabotオープンfeat
難易度 2/5 1時間未満 初心者へのやさしさ 72/100
open-nudge/opentemplate#204 ·
メンテナーはふだん 1 日以内に返信
-
omarchy-brightness-keyboard: double idle-blank saves brightness 0, so wake 'restore' keeps the keyboard backlight off対応中かも このイシューにリンクされたプルリクエストがオープン中、またはマージ済みです。 オープン
難易度 2/5 1〜3時間 初心者へのやさしさ 82/100
メンテナーはふだん 1 日以内に返信
-
Type/Improvement
難易度 2/5 1〜3時間 初心者へのやさしさ 62/100
wso2/product-integrator-mi#5099 ·
メンテナーはふだん 1 日以内に返信
-
[Bug] Windows 上 pwsh 工具名未按 exec 分类着色，与状态圆点颜色家族分裂再び着手できるかも このイシューのプルリクエストはマージされずにクローズされました。 オープン
難易度 2/5 1〜3時間 初心者へのやさしさ 62/100
ccch1mneyyy/dsh-TUI#1555 ·
メンテナーはふだん 1 日以内に返信
-
Standalone Unix updater reports success when curl fails (`curl | sh` masks the exit status); Windows path is unaffected対応中かも @akhil9tiet が今日担当しました。 オープンbug CLI windows-os
難易度 2/5 1〜3時間 初心者へのやさしさ 72/100
openai/codex#54465 · コメント 1 件 ·
メンテナーはふだん 1 日以内に返信