Colons checker in CI. #125

Merged
mdk merged 7 commits from mdk/python-docs-fr:mdk-colons into 3.11 2023-04-14 11:16:52 +00:00
Owner
No description provided.
mdk force-pushed mdk-colons from 4ebe983bbd to 0cdfce7d26 2023-04-08 13:28:23 +00:00 Compare
mdk force-pushed mdk-colons from 289165b30b to 6447db27cc 2023-04-08 14:28:53 +00:00 Compare
mdk force-pushed mdk-colons from 6447db27cc to 4d58534c76 2023-04-08 14:37:08 +00:00 Compare
mdk force-pushed mdk-colons from 4d58534c76 to fae0d5849c 2023-04-08 14:39:12 +00:00 Compare
mdk force-pushed mdk-colons from fae0d5849c to 275b4a99ca 2023-04-08 14:46:23 +00:00 Compare
mdk force-pushed mdk-colons from 275b4a99ca to fc30b465f9 2023-04-08 14:46:50 +00:00 Compare
Collaborator

Ce script et padpo ne sont pas d'accord sur les : ::

Ce script et *padpo* ne sont pas d'accord sur les *: ::*
Author
Owner

Ce script et padpo ne sont pas d'accord sur les : ::

https://github.com/AFPy/padpo/issues/67 :(

> Ce script et *padpo* ne sont pas d'accord sur les *: ::* https://github.com/AFPy/padpo/issues/67 :(
Author
Owner

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ù.

> 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 to 6799835987 2023-04-10 13:29:04 +00:00 Compare
Collaborator

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).
Collaborator

avec la pj.

avec la pj.
Author
Owner

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 14:31:02 +00: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
Collaborator

limit check to $SRC after this commit?
same for long lines check?

limit check to $SRC after this commit? same for long lines check?
Author
Owner

Super idée, c'est fait.

Super idée, c'est fait.
mdk marked this conversation as resolved
ChristopheNan approved these changes 2023-04-13 20:01:50 +00:00
mdk force-pushed mdk-colons from c900f9b587 to fbc93b968c 2023-04-14 11:14:03 +00:00 Compare
mdk merged commit 3697e62ab7 into 3.11 2023-04-14 11:16:52 +00:00
mdk referenced this issue from a commit 2023-04-14 11:16:52 +00:00
Sign in to join this conversation.
No reviewers
No Milestone
No project
No Assignees
2 Participants
Notifications
Due Date
The due date is invalid or out of range. Please use the format 'yyyy-mm-dd'.

No due date set.

Dependencies

No dependencies set.

Reference: AFPy/python-docs-fr#125
No description provided.