Colons checker in CI.#125

J'ai fait quelques ajustements, et je ne trouve pas d'incohérence entre padpo et ce script, tu en vois encore ? Si oui je veux bien que tu me dise où.

> Ce script et *padpo* ne sont pas d'accord sur les *: ::* J'ai fait quelques ajustements, et je ne trouve pas d'incohérence entre padpo et ce script, tu en vois encore ? Si oui je veux bien que tu me dise où.

mdk force-pushed mdk-colons from dc29cf3e68

ci/woodpecker/pr/woodpecker Pipeline was successful

Details

to 6799835987

ci/woodpecker/pr/woodpecker Pipeline failed

Details

2023-04-10 15:29:04 +02:00

Compare

ChristopheNan commented

2023-04-10 18:31:08 +02:00

Merci pour ce vérificateur Ça me semble cohérent avec padpo.

La vérification dans le Makefile est un peu brute, une fois que cette PR sera passée. Le glob du vérificateur est taillé pour la structure de l'arborescence de la traduction, je proposerais (voir pj) d'avoir la liste des fichiers à vérifier dans la ligne de commande (sans chercher de récursion par le programme, celle-ci étant fait en amont au besoin).

Merci pour ce vérificateur Ça me semble cohérent avec *padpo*. La vérification dans le *Makefile* est un peu brute, une fois que cette PR sera passée. Le *glob* du vérificateur est taillé pour la structure de l'arborescence de la traduction, je proposerais (voir pj) d'avoir la liste des fichiers à vérifier dans la ligne de commande (sans chercher de récursion par le programme, celle-ci étant fait en amont au besoin).

ChristopheNan commented

2023-04-10 18:41:49 +02:00

avec la pj.

check-colon.diff.txt

3.4 KiB

mdk commented

2023-04-10 22:34:54 +02:00

Ah top :))

En réorganisant un peu le Makefile ça donne :

$ make verifs Checking all files using .scripts/line-length.py... Checking all files using .scripts/check-colon.py... Checking all files using sphinx-lint... No problems found.

Je triche, c'est sphinx-lint qui emmet le 'No problems found.' :D

Ah top :)) En réorganisant un peu le Makefile ça donne : ```text $ make verifs Checking all files using .scripts/line-length.py... Checking all files using .scripts/check-colon.py... Checking all files using sphinx-lint... No problems found. ``` Je triche, c'est sphinx-lint qui emmet le 'No problems found.' :D

ChristopheNan reviewed

2023-04-13 16:31:02 +02:00

Makefile Outdated

              @ -188,0 +188,4 @@  
     .PHONY: check-colons  
     check-colons:  
     @echo "Checking all files using .scripts/check-colon.py..."  
     @python .scripts/check-colon.py --check *.po */*.po  
  