pr-pipeline: apply-patch held() honours a Hold: line added to an adopted owner PR mid-run
メンテナーはふだん 1 日以内に返信
まだ誰も着手していません。
評価
調査の方向性
まず apply-patch.mjs から始めて、180～186行あたりの held() を読み、その後、#8997 で説明されているオーナー作成 PR の処理を調べてください。オーナー作成 PR に dispatch と push の間に Hold: 行が追加されるテストを追加してください。その実行でその PR が push されず、既存の Lane App PR の hold 動作が維持されれば完了です。
索引モデルが issue の本文から書いたものです。
説明
Refs: #8997
apply-patch.mjs held() (~:180-186) re-checks a PR's draft state and hold labels at push time, but not a body line starting Hold:. Since #8997, fix-ci adopts idle owner PRs and skips one that carries a Hold: line at dispatch; a Hold: line added while a fix-ci run is already going does not stop that run's push. The next dispatch respects it.
Fix: have held() read the PR body and treat a line starting Hold: as held, for adopted owner PRs (lanes App PRs carry Hold: a human merges this, which holds the merge, not a CI fix, so scope the check to owner-authored PRs as #8997 does). Test that fails first: an owner PR whose body gains a Hold: line between dispatch and push is not pushed.
From the S&C review of #8997 (security OK at 69983e27; this was the accepted residual).
- 主要言語
- Shell
- スター
- 22
- フォーク
- 2
- 平均マージ
- 6時間 58分
- マージ済み PR（30日）
- 951
環境構築
- Dockerfile・Docker Compose ファイルなし
- プルリクエストのテンプレートなし
- コントリビューションガイドを読む
はじめの一歩
- issue を最後まで読み、次にプロジェクトのコントリビューションガイドを読みます。
- 着手することを issue にコメントします — 二人が同じ作業をするのを防げます。
- リポジトリをフォークし、ブランチを切って変更します。
- issue 番号を参照したプルリクエストを送ります。
melodic-software/claude-code-plugins のほかの issue
-
priority: medium status: needs-decision work-class: scoped
難易度 2/5 1〜3時間 初心者へのやさしさ 78/100
melodic-software/claude-code-plugins#9339 · コメント 2 件 ·
メンテナーはふだん 1 日以内に返信
-
pipeline-canary priority: low status: ready work-class: mechanical
難易度 1/5 1時間未満 初心者へのやさしさ 88/100
melodic-software/claude-code-plugins#9263 · コメント 1 件 ·
メンテナーはふだん 1 日以内に返信
-
good first issue priority: low
難易度 1/5 1時間未満 初心者へのやさしさ 90/100
melodic-software/claude-code-plugins#9251 · コメント 1 件 ·
メンテナーはふだん 1 日以内に返信
-
area: security needs-triage priority: low
難易度 2/5 1〜3時間 初心者へのやさしさ 79/100
melodic-software/claude-code-plugins#9211 · コメント 1 件 ·
メンテナーはふだん 1 日以内に返信
-
priority: low
難易度 2/5 1〜3時間 初心者へのやさしさ 68/100
melodic-software/claude-code-plugins#9202 · コメント 1 件 ·
メンテナーはふだん 1 日以内に返信
melodic-software/claude-code-plugins の issue をすべて見る
似ている issue
-
難易度 2/5 1〜3時間 初心者へのやさしさ 72/100
MISP/misp-docker#478 ·
メンテナーはふだん 1 日以内に返信
-
roadmap-100
難易度 2/5 1〜3時間 初心者へのやさしさ 80/100
メンテナーはふだん 1 日以内に返信
-
Theme switch hangs in omarchy-theme-set-browser when Brave Origin runs and brave-bin is installedオープン
難易度 2/5 1〜3時間 初心者へのやさしさ 78/100
メンテナーはふだん 1 日以内に返信
-
数据界面的横向滚动条缺失オープン
難易度 2/5 1〜3時間 初心者へのやさしさ 62/100
zilliztech/attu#1037 ·
-
bug
難易度 2/5 1〜3時間 初心者へのやさしさ 78/100
community-scripts/ProxmoxVE#18073 · コメント 1 件 ·
メンテナーはふだん 1 日以内に返信