Hacktoberfest 2026 : les issues que les mainteneurs ont marquées pour octobre, ouvertes et accessibles aux débutants. Parcourir les issues Hacktoberfest

pr-pipeline: clear a feedback-not-landed escalation once the head moves and no lane thread is open

Fermée Adaptée aux débutants
  • Quelque chose ne va pas ?
#10,074 4 commentaires 0 réactions 0 personnes assignées Voir sur GitHub

Les mainteneurs répondent en général sous 1 jour

@melodic-automation-lanes y travaille déjà.

Depuis le 29/10/2026.

  • #10886 par @melodic-automation-lanes — ouverte

Évaluation

Difficulté
2/5
Temps estimé
1-3 heures
Accessibilité débutants
83/100
Type d'issue
Bug
Clarté
Clairement spécifiée
Activité
Active
Stack technique
javascript
Domaine
devtools

Piste de recherche

Lisez .github/scripts/pr-merge/clear-escalations.mjs, en particulier le tableau CHECKS, ainsi que les cas associés de clear-escalations.test.mjs. Ajoutez la vérification demandée à l’aide de la clé de reprise de l’escalade et des fils de revue non résolus du lane, puis exécutez ce fichier de test. Le travail est terminé lorsque le déplacement du head sans fil ouvert dans le lane efface l’escalade, tandis qu’un head inchangé ou un fil ouvert dans le lane la maintient.

Rédigé par le modèle d'indexation à partir du texte de l'issue.

Description

priority: medium status: needs-decision work-class: scoped

Problem

clear-escalations.mjs clears a pr-unblock escalation once its reason no longer holds, but its CHECKS table (.github/scripts/pr-merge/clear-escalations.mjs:120-132) covers only workflow-staged, unresolved-conflict, protected-path-conflict and feedback-protected-path. A feedback-not-landed escalation stays labeled after the cause is gone.

Example: #9965 escalated feedback-not-landed at resume key e5183af3; the head moved to 898665b3 with no unresolved review thread left, and ci-status stayed red only on the needs-human label until a person removed it.

Owner decision (2026-10-25)

A stop is worth a person only for security, intent, or a real charge. feedback-not-landed is none of those once the head has moved.

Fix

Add a feedback-not-landed entry to CHECKS in clear-escalations.mjs: the reason no longer holds when the head differs from the escalation's resume key and no review thread the lane replied to is left unresolved. Reuse the existing clearing path; add no second mechanism.

Done when

  • clear-escalations.test.mjs covers: head moved and no open lane thread clears; head unchanged keeps; an open lane thread keeps.
Langage dominant
Shell
Étoiles
22
Forks
2
Merge moyen
2 h 40 min
PR mergées (30 j)
936

Préparer son environnement

Par où commencer

  1. Lisez l'issue en entier, puis le guide de contribution du projet.
  2. Signalez en commentaire que vous la prenez — cela évite que deux personnes fassent le même travail.
  3. Forkez le dépôt et travaillez sur une branche.
  4. Ouvrez une pull request qui référence le numéro de l'issue.

Autres issues de melodic-software/claude-code-plugins

Toutes les issues de melodic-software/claude-code-plugins

Issues similaires

Plus d'issues Shell/Bash

Recevez les nouvelles issues par e-mail

Un résumé court des issues GitHub adaptées aux débutants.