Openwisp-utils: [qa] La vérification du message de validation échoue s'il y a un point après le problème

Créé le 7 mai 2021  ·  9Commentaires  ·  Source: openwisp/openwisp-utils

Tous les 9 commentaires

Puis-je faire ce problème

Bien sûr @keshavgbpecdelhi vas-y

@devkapilbansal pouvez-vous dire

Je pense que vous devez corriger cette fonction
https://github.com/openwisp/openwisp-utils/blob/1f73cc3d522478cd34bd6f10d3dfa98be12e4263/openwisp_utils/qa.py#L83

J'ai essayé d'examiner ce problème et j'ai découvert que l'expression régulière renvoyait #344. de la description longue qui est ensuite associée à #344 de la description courte et les tests ont échoué car #344. n'est pas le même que #344

Doit-on ajouter "." dans le cas du saut ?
skip_cases = [
# fusions
r'^Merge pull request #[A-Za-z0-9.]* from',
r"^Fusionner la branche '(. ?)' dans",# sortiesr'^[A-Za-z0-9.] release$',
r"^Bumped VERSION à (. ?)$",r"^Bump (. ?) de (. ?) à (. ?)$",
]

C'est pour une brève description seulement

@keshavgbpecdelhi avez-vous essayé de parcourir le code une fois et de comprendre ce qui se passe là-bas ? La meilleure façon de corriger l'erreur est d'essayer de la reproduire localement, de voir quelle partie du code est responsable de cela, d'écrire un scénario de test qui échouera à coup sûr à moins que l'erreur ne soit corrigée, puis de résoudre l'erreur.

@keshavgbpecdelhi la première étape consiste à écrire un test d'échec pour reproduire le problème, voir https://github.com/openwisp/openwisp-utils/blob/master/tests/test_project/tests/test_qa.py#L79 -L135.

Cette page vous a été utile?
0 / 5 - 0 notes