Openwisp-utils: [qa] Überprüfung der Commit-Nachricht schlägt fehl, wenn nach dem Problem ein Punkt erscheint

Erstellt am 7. Mai 2021  ·  9Kommentare  ·  Quelle: openwisp/openwisp-utils

Alle 9 Kommentare

Kann ich dieses Problem machen?

Klar @keshavgbpecdelhi mach weiter

@devkapilbansal können Sie sagen, wo Änderungen vorgenommen werden müssen (welcher Code geändert werden musste)

Ich denke, Sie müssen diese Funktion reparieren
https://github.com/openwisp/openwisp-utils/blob/1f73cc3d522478cd34bd6f10d3dfa98be12e4263/openwisp_utils/qa.py#L83

Ich habe versucht, mir dieses Problem anzusehen und festgestellt, dass Regex #344. aus der langen Beschreibung zurückgibt, die dann mit #344 aus der kurzen Beschreibung abgeglichen wird und die Tests fehlgeschlagen sind, weil #344. nicht gleich ist wie #344

Müssen wir "." hinzufügen? im Fall des Überspringens?
skip_cases = [
# fusioniert
r'^Pull-Request #[A-Za-z0-9.]* zusammenführen von',
r"^Zweig '(. ?)' zusammenführen in",# Veröffentlichungenr'^[A-Za-z0-9.] loslassen$',
r"^Version auf (. ?)$ geändert
?) nach (. ?)$",
]

Es dient nur der kurzen Beschreibung

@keshavgbpecdelhi hast du einmal versucht, den Code zu durchsuchen und zu verstehen, was dort passiert? Der beste Weg, den Fehler zu beheben, besteht darin, ihn lokal zu reproduzieren, zu sehen, welcher Teil des Codes dafür verantwortlich ist, einen Testfall zu schreiben, der mit Sicherheit fehlschlägt, es sei denn, der Fehler wird behoben, und dann den Fehler zu beheben.

@keshavgbpecdelhi Der erste Schritt besteht darin, einen fehlgeschlagenen Test zu schreiben, um das Problem zu replizieren, siehe https://github.com/openwisp/openwisp-utils/blob/master/tests/test_project/tests/test_qa.py#L79 -L135.

War diese Seite hilfreich?
0 / 5 - 0 Bewertungen