Hacktoberfest 2026: die Issues, die Maintainer für den Oktober markiert haben – offen und einsteigerfreundlich. Hacktoberfest-Issues durchsuchen

Let's talk about the CI situation

Offen
  • Stimmt etwas nicht?
#1,614 20 Kommentare 8 Reaktionen 0 zugewiesene Personen Auf GitHub ansehen

Dieses Issue hat noch niemand übernommen.

Bewertung

Schwierigkeit
5/5
Geschätzter Aufwand
Über eine Woche
Anfängerfreundlichkeit
25/100
Issue-Typ
Feature
Klarheit
Muss geklärt werden
Aktivitätsstatus
Veraltet
Tech-Stack
node.js
Bereich
ci-cd, testing-qa

Rechercherichtung

Es werden keine Quelldateien oder Tests genannt. Lies zunächst diese Diskussion und das referenzierte nodejs/build issue 3887, und prüfe dann den Kommentar-Thread auf eine vereinbarte Richtung zur Erkennung und Behandlung von fehlerhaften CI-Tests. Als erledigt würde ein konkreter, akzeptierter Plan gelten, nicht eine spezifische Codeänderung.

Vom Indexierungsmodell aus dem Issue-Text verfasst.

Beschreibung

As of recently, especially with https://github.com/nodejs/build/issues/3887, dealing with the CI when preparing a release or when trying to land a PR has become very frustrating. I tried to list all the pain points, let me know if I forgot something:

  • The CI is so flaky it's systematically takes several attempts to get a passing one.
  • Most folks do not bother to check the failures.
  • Even fewer folks are reporting flaky tests (I plead guilty).
  • Even fewer folks are investigating those flakes (and I get it, the very nature of flaky tests makes them very hard to reproduce, let alone debugging them).
  • Opening a PR to mark test(s) as flaky often receives objections that the flakiness should be solved instead of ignored (which I totally get, those flakes are most of the time due to a bug in node; but it also makes the situation more frustrating for contributing, and also for other projects that build their own node and expect tests not marked as flaky to not be flaky).

I think we need to discuss:

  1. Is there a way to better detect flaky tests before landing them?
  2. How can we handle the immediate situation?

I'm opening this in the TSC repo, because it's kind of a meta discussion that I'm guessing is not going to be of much interest to folks following nodejs/node, but of course anyone is welcome to participate to the discussion.

Vorherrschende Sprache
JavaScript
Sterne
693
Forks
142
Ø Merge
8 T. 7 Std.
Gemergte PRs (30 T.)
3

Entwicklungsumgebung

Erste Schritte

  1. Lesen Sie das ganze Issue und danach den Beitragsleitfaden des Projekts.
  2. Schreiben Sie ins Issue, dass Sie es übernehmen — das erspart doppelte Arbeit.
  3. Forken Sie das Repository und arbeiten Sie in einem Branch.
  4. Öffnen Sie einen Pull Request, der die Issue-Nummer nennt.

Mehr aus nodejs/TSC

Alle Issues in nodejs/TSC

Ähnliche Issues

Weitere Issues zu JavaScript

Neue Issues direkt in Ihr Postfach

Eine kurze Übersicht über anfängerfreundliche GitHub-Issues.