pr-pipeline: apply-patch held() honours a Hold: line added to an adopted owner PR mid-run
I maintainer di solito rispondono entro 1 giorno
Nessuno ha ancora preso questa issue.
Valutazione
- Difficoltà
- 2/5
- Tempo stimato
- 1-3 ore
- Idoneità per principianti
- 73/100
Direzione di ricerca
Inizia con apply-patch.mjs e leggi held() intorno alle righe 180–186, poi esamina la gestione delle PR create dai proprietari descritta in #8997. Aggiungi un test in cui una PR creata da un proprietario riceve una riga Hold: tra il dispatch e il push; il lavoro è completato quando quell'esecuzione non fa push della PR e il comportamento esistente di hold per le PR di Lane App rimane intatto.
Scritto dal modello di indicizzazione a partire dal testo della issue.
Descrizione
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).
- Lingua principale
- Shell
- Stelle
- 22
- Fork
- 2
- Merge medio
- 7h 23m
- PR unite (30g)
- 950
Preparare l'ambiente
- Nessun Dockerfile né file Docker Compose
- Nessun modello di pull request
- Leggi la guida per i contributori
Come iniziare
- Leggi tutta la issue e poi la guida ai contributi del progetto.
- Commenta sulla issue per dire che te ne occupi tu — evita che due persone facciano lo stesso lavoro.
- Fai un fork del repository e lavora su un branch.
- Apri una pull request che faccia riferimento al numero della issue.
Altre issue di melodic-software/claude-code-plugins
-
fix(session-flow): redact.py misses bare sk-ant-oat01- tokens and stateless ghs_ tokens with dotsApertaneeds-human
Difficoltà 2/5 1-3 ore Idoneità per principianti 82/100
melodic-software/claude-code-plugins#9026 · 1 commento ·
I maintainer di solito rispondono entro 1 giorno
-
needs-human
Difficoltà 2/5 1-3 ore Idoneità per principianti 76/100
melodic-software/claude-code-plugins#9013 · 1 commento ·
I maintainer di solito rispondono entro 1 giorno
-
needs-triage
Difficoltà 2/5 1-3 ore Idoneità per principianti 68/100
melodic-software/claude-code-plugins#9000 · 3 commenti ·
I maintainer di solito rispondono entro 1 giorno
-
fix(source-control): ready hold check treats hyphenated identifiers containing wip as a WIP markerApertapriority: medium
Difficoltà 2/5 1-3 ore Idoneità per principianti 77/100
melodic-software/claude-code-plugins#8977 · 1 commento ·
I maintainer di solito rispondono entro 1 giorno
-
fix(ai-slop): detect.sh --paths-file drops the last entry when the file has no trailing newlineApertapriority: low
Difficoltà 2/5 1-3 ore Idoneità per principianti 86/100
melodic-software/claude-code-plugins#8974 · 1 commento ·
I maintainer di solito rispondono entro 1 giorno
Tutte le issue di melodic-software/claude-code-plugins
Issue simili
-
enhancement
Difficoltà 2/5 1-3 ore Idoneità per principianti 65/100
-
area:terraform priority:low source:reader type:enhancement
Difficoltà 2/5 1-3 ore Idoneità per principianti 65/100
stephrobert/blog-roadmap#104 ·
-
Difficoltà 2/5 1-3 ore Idoneità per principianti 65/100
ReproNim/containers#172 ·
-
Difficoltà 2/5 1-3 ore Idoneità per principianti 67/100
fairagro/m4.2_infrastructure#54 · 1 commento ·
I maintainer di solito rispondono entro 1 giorno
-
Difficoltà 2/5 1-3 ore Idoneità per principianti 78/100
I maintainer di solito rispondono entro 1 giorno