nodejs / nodejs/node-core-utils
Cancelled GitHub CI rerun after success is treated as failure
Dieses Issue hat noch niemand übernommen.
- Vorherrschende Sprache
- JavaScript
- Sterne
- 314
- Forks
- 137
- Ø Merge
- 2 T. 8 Std.
- Gemergte PRs (30 T.)
- 24
Beschreibung
PR: https://github.com/nodejs/node/pull/43085
Action rerun attempt: https://github.com/nodejs/node/actions/runs/2320332585/attempts/7
git-node land output: https://github.com/nodejs/node/pull/43085#issuecomment-1126858386
At the moment of landing, the latest run of that action was cancelled, there were 5 prior successful runs, and GitHub UI reported everything as green.
However, ncu treated it as Last GitHub CI failed error, which is either wrong (not sure if easily fixable, as it requires failproof checking previous attempts) or confusing (Last GitHub CI cancelled might be a better error message).
Details: in PRChecker.checkGitHubCI(), commit.checkSuites.nodes[2] for this case was
{
app: { slug: 'github-actions' },
conclusion: 'CANCELLED',
status: 'COMPLETED'
}
Beitragsleitfaden
Erste Schritte
- Lies das ganze Issue und danach den Beitragsleitfaden des Projekts.
- Schreib ins Issue, dass du es übernimmst — das erspart doppelte Arbeit.
- Forke das Repository und arbeite in einem Branch.
- Öffne einen Pull Request, der die Issue-Nummer nennt.
Rechercherichtung
Beginne in PRChecker.checkGitHubCI() und untersuche, wie commit.checkSuites.nodes interpretiert wird, wobei du den verknüpften PR, den Action-Versuch und die git-node-land-Ausgabe als Reproduktionskontext verwendest. Überprüfe das Verhalten, wenn der neueste GitHub-Actions-Lauf nach früheren erfolgreichen Läufen abgebrochen wird. Die Aufgabe ist erledigt, wenn der gemeldete CI-Status und die Fehlermeldung diesen Fall korrekt widerspiegeln und Tests dafür vorhanden sind.
Vom Indexierungsmodell aus dem Issue-Text verfasst.
Bewertung
- Tech-Stack
- javascript
- Bereich
- cli
- Issue-Typ
- Bug
- Schwierigkeit
- 3/5
- Geschätzter Aufwand
- 1-2 Tage
- Aktivitätsstatus
- Ruhig
- Klarheit
- Größtenteils klar
- Anfängerfreundlichkeit
- 55/100