Automatic detection of "Fixes" should be stricter

Offen

#592 erstellt am 11.11.2021 1 Kommentar 0 Reaktionen 0 zugewiesene Personen Auf GitHub ansehen

Dieses Issue hat noch niemand übernommen.

Bewertung

Schwierigkeit: 3/5
Geschätzter Aufwand: 1-2 Tage
Anfängerfreundlichkeit: 42/100

Issue-Typ: Bug
Klarheit: Größtenteils klar
Aktivitätsstatus: Veraltet
Tech-Stack: javascript, node.js
Bereich: tooling

Rechercherichtung

Verwende PR #40613 und die in diesem Issue angezeigten generierten Metadaten als Reproduktionsfall; verfolge den Einstiegspunkt von node-core-utils, der Fixes-Metadaten aus Commit-Nachrichten erkennt. Als abgeschlossen gilt die Aufgabe, wenn Prosa, die „fix“ erwähnt, keine inkorrekte Fixes-Zeile mehr erzeugt, während gültige Fixes-Referenzen weiterhin erkannt werden.

Vom Indexierungsmodell aus dem Issue-Text verfasst.

Beschreibung

bug

In https://github.com/nodejs/node/pull/40613, node-core-utils@1.30.0 incorrectly adds a Fixes: metadata line:

----------------------------------- PR info ------------------------------------
Title      crypto: fix build without scrypt (#40613)
Author     Martin 'JaMa' Jansa <Martin.Jansa@gmail.com> (@shr-project, first-time contributor)
Branch     shr-project:jansa/scrypt -> nodejs:master
Labels     crypto, c++, author ready
Commits    1
 - crypto: fix build without scrypt
Committers 1
 - Martin Jansa <martin.jansa@lge.com>
PR-URL: https://github.com/nodejs/node/pull/40613
Fixes: https://github.com/In
Reviewed-By: Tobias Nießen <tniessen@tnie.de>
Reviewed-By: Voltrex <mohammadkeyvanzade94@gmail.com>
Reviewed-By: Anna Henningsen <anna@addaleax.net>
------------------------------ Generated metadata ------------------------------
PR-URL: https://github.com/nodejs/node/pull/40613
Fixes: https://github.com/In
Reviewed-By: Tobias Nießen <tniessen@tnie.de>
Reviewed-By: Voltrex <mohammadkeyvanzade94@gmail.com>
Reviewed-By: Anna Henningsen <anna@addaleax.net>
--------------------------------------------------------------------------------

The commit message was:

crypto: fix build without scrypt

* add missing semicolon to fix:
  In file included from ../src/node_crypto.h:47,
                   from ../src/node.cc:46:
  ../src/crypto/crypto_scrypt.h:80:2:
  error: expected ';' after struct definition
     80 | }
        |  ^
        |  ;

  and fix typo in the comment

Signed-off-by: Martin Jansa <martin.jansa@lge.com>

_{(Original post in https://github.com/nodejs/node/pull/40613#issuecomment-966507251.)}

Vorherrschende Sprache: JavaScript
Sterne: 313
Forks: 137
Ø Merge: 2 T. 8 Std.
Gemergte PRs (30 T.): 24

Beitragsleitfaden

Beitragsleitfaden öffnen

Erste Schritte

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

Mehr aus nodejs/node-core-utils

git node wpt <name> crashes if <name> isn't found Offen

bug

Schwierigkeit 2/5 1-3 Stunden Anfängerfreundlichkeit 62/100

nodejs/node-core-utils#664 · 1 Kommentar · Erstellt am 06.12.2022
ncu-ci walk: exclude PR-modified tests from the flaky count Offen

Schwierigkeit 3/5 1-2 Tage Anfängerfreundlichkeit 65/100

nodejs/node-core-utils#1164 · Erstellt am 15.8.2026
checkGitHubCI blocks indefinitely on orphaned check suites stuck in QUEUED Offen

Schwierigkeit 4/5 3-5 Tage Anfängerfreundlichkeit 68/100

nodejs/node-core-utils#1160 · 3 Kommentare · Erstellt am 10.8.2026
pr_checker: ignore superseded GitHub Actions check suites when a head SHA is reused Offen

Schwierigkeit 3/5 1-2 Tage Anfängerfreundlichkeit 68/100

nodejs/node-core-utils#1151 · Erstellt am 08.8.2026
Add large pull request checks in `git node land` Offen

Schwierigkeit 5/5 Über eine Woche Anfängerfreundlichkeit 35/100

nodejs/node-core-utils#1063 · Erstellt am 24.4.2026

Alle Issues in nodejs/node-core-utils