The PR needs to be merged I think to be ran

The PR needs to be merged I think to be ran

no.

no.

Yeah syntax is better

Pour les débutants, c'est peut être mieux présenté comme ça qu'avec Travis.

Il reste un petit souci avec padpo, je n'ai pas trop enquêté encore, cc @vpoulailleau : vpoulailleau/pygrammalecte#1 (qui semble d'ailleurs exit(0) quand ça arrive ?)

@JulienPalard Tu as des titres d'actions en anglais et en francais, faut que on se décide ;)

(qui semble d'ailleurs exit(0) quand ça arrive ?)

Oops… Je viens de créer vpoulailleau/pygrammalecte#2

Et je t'ai répondu dans vpoulailleau/pygrammalecte#1

Tout passe, on peut tenter de merger ça ? Je trouve ça plus lisible que Travis.

Tout passe, on peut tenter de merger ça ? Je trouve ça plus lisible que Travis.

Ok ! On laisse travis pour l'instant ?

PS: C'est aussi grave plus rapide non ?

Moi aussi je trouve ça plus lisible.
Par contre, pour padpo, le build est toujours OK, même s'il y a des warnings/erreurs. Ça pourrait être bien que le test/Grammaire soit KO (un exit d'erreur), mais pas requis pour merger la pull request. Je ne sais pas si on peut faire des actions facultatives. Vous savez ?

On peut faire des actions facultatives :)

(J'peux pas merge si personne approuve, alors, que si quelqu'un approuve, qu'il appuie sur le foutu bouton.)

On peut faire des actions facultatives :)

@JulienPalard Ça n'a pas l'air si simple : actions/runner#2347, tu vois un moyen de faire en sorte que le check Grammaire fail sans que ça marque la PR comme failed ? Si oui, padpo fait déjà un sys.exit(1) s'il détecte des erreurs, on pourrait donc failer le check (voui, ça fait un peu franglais tout ça 😉)

    errors, warnings = check_paths(path, pull_request_info=pull_request_info)
    if errors:
        sys.exit(1)

@JulienPalard Ça n'a pas l'air si simple : actions/runner#2347, tu vois un moyen de faire en sorte que le check Grammaire fail sans que ça marque la PR comme failed ?

Oui dans les settings du repo on peut choisir quel test peut ne pas réussir tout en autorisant le merge.

Tu es sûr que ça exit 1 @vpoulailleau https://github.com/python/python-docs-fr/runs/1444350732 ?

@JulienPalard Je viens de tester pour confirmer, oui padpo exit 1

Par contre, la commande PATH=$PATH:$PWD/.venv/bin/ padpo -i *.po */*.po 2>&1 | grep -v -Ff padpo.ignore termine par grep qui fait exit 0 !

La page de man de grep dit :

EXIT STATUS
Normally the exit status is 0 if a line is selected, 1 if no lines were selected, and 2 if an error occurred. However, if the -q or --quiet or --silent is used and a line is selected, the exit status is 0 even if an error occurred.

Il faut du coup récupérer le exit code de padpo puis faire le grep puis retourner le exit code.

Par contre, la commande PATH=$PATH:$PWD/.venv/bin/ padpo -i *.po */*.po 2>&1 | grep -v -Ff padpo.ignore termine par grep qui fait exit 0 !

AHHH mais ouiiiiiiiiiiiiiiiiiii !

Bon bah on va laisser ça comme ça en attendant, vu qu'on veut que padpo soit toujours vert pour le moment si j'ai bien suivi ? :p