nodejs / nodejs/core-validate-commit

Invalid Commit Message

Offen
#117 1 Kommentar 0 Reaktionen 0 zugewiesene Personen Auf GitHub ansehen

Dieses Issue hat noch niemand übernommen.

Vorherrschende Sprache
JavaScript
Sterne
55
Forks
62
Ø Merge
4 T. 9 Std.
Gemergte PRs (30 T.)
1

Beschreibung

This PR was merged with a non-imperative verb starting the commit message: benchmark: added new benchmarks for blob.

From what the docs says, it should be a invalid message.

The CI did raised an error about the length (that I fix) but didn't raise an error about this verb and it was merged.

Beitragsleitfaden

Beitragsleitfaden öffnen

Erste Schritte

  1. Lies das ganze Issue und danach den Beitragsleitfaden des Projekts.
  2. Schreib ins Issue, dass du es übernimmst — das erspart doppelte Arbeit.
  3. Forke das Repository und arbeite in einem Branch.
  4. Öffne einen Pull Request, der die Issue-Nummer nennt.

Rechercherichtung

Beginne mit den Richtlinien für Commit-Nachrichten und der durch PR 49730 abgedeckten Validierung und verfolge dann die Prüfung zurück, die den Fehler zur Nachrichtenlänge gemeldet hat. Überprüfe, wie das einleitende Verb des Commit-Betreffs ausgewertet wird. Als erledigt gilt die Aufgabe, wenn ein nicht imperativer Betreff wie „added new benchmarks for blob“ von CI abgelehnt wird, während gültige imperative Formulierungen bestehen.

Vom Indexierungsmodell aus dem Issue-Text verfasst.

Bewertung

Tech-Stack
javascript
Bereich
ci-cd, testing-qa
Issue-Typ
Bug
Schwierigkeit
3/5
Geschätzter Aufwand
1-2 Tage
Aktivitätsstatus
Veraltet
Klarheit
Größtenteils klar
Anfängerfreundlichkeit
35/100

Neue Issues direkt in Ihr Postfach

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