Github action pour traduire vos plugins 🌐

[Noyax37]

merci @nebz

[Michel_F]

Est-ce que la solution suivante fonctionnerait ? C’est celle que j’utilise en prévision des traductions…

sprintf(__("blabla %s machin chouette", __FILE__), $truc);

Ca permet de placer la variable où il faut dans toutes les langues.
En fait il suffirait que deepl le supporte et c’est bon, si j’ai bien compris.

[Mips]

Si c’est ça qu’il faut faire ainsi la phrase reste complète
Surtout que lorsqu’on traduit il peut y avoir des inversions de mots

[Noyax37]

j’essaierai bien mais rien que pour mon 1er plugin j’en suis à 300 000 caractères traduits sur les 500 000 possibles gratuitement… J’en garde un peu pour les autres

[Mips]

@Noyax37

l’action que d’une façon manuelle et pas automatique?

Si c’est possible
Mais pourquoi ne pas juste valider la pr?
Ainsi il ne traduira plus ce qui est déjà traduit

[nebz]

@Michel_F

sprintf(__("blabla %s machin chouette", __FILE__), $truc);

par contre pas possible en html du coup… ou alors tu dois passer en php rien que pour ça … bof

[Mips]

Oui, ou formuler autrement

[Michel_F]

Effectivement, juste en php

[Noyax37]

@Mips

Mais pourquoi ne pas juste valider la pr?

Oui c’est ce que je fais mais comme je m’aperçois qu’il y a des pb à certains endroits où que j’ai oublié des trucs bah ça relance presque tout à chaque fois

Pour les prochains je ferai plus attention

[nebz]

@Mips

Mais pourquoi ne pas juste valider la pr?

tiens à propos, tu avais parlé d’ajouter un paramètre pour un auto-merge et delete branch.

je remarque après plusieurs mois d’utilisation que à chaque fois que j’ai vérifié dans les autres langues ça semblait cohérent, donc au final je vérifie même plus et je ne fais que merger et delete la branch … donc je fais confiance à 100%, pour moi un tel param serait sympa

[Mips]

@Noyax37

ça relance presque tout à chaque fois

Non il ne fait plus d’appel a deepl si c’est traduit

[Mips]

@nebz

d’ajouter un paramètre pour un auto-merge

Oui mais ca n’existe pas sur l’action utilisée actuellement
Donc c’était un peu plus compliqué à mettre en place.
C’est un peu loin dans ma tête, faudrait que je revois cette partie

[Noyax37]

sauf si tu changes ce qui est à traduire

[Mips]

Pour désactiver l’automatisation il suffit de retirer la partie push: dans la section on: (ce sont les triggers)
Workflow_dispatch: c’est pour pouvoir le lancer manuellement.
Voir doc des workflow github pour plus de détails

[nebz]

ah par contre j’ai juste ajouté un espace en fin de chaine pour tester et je vois que pas mis dans les langues, tu fais un trim de ton coté ou c’est deepl ?

[Mips]

aucun trim, d’ailleurs on le voit dans la « clé » … mais je devrais peut-être

[nebz]

oui on en revient aux bonnes pratiques

__("Ceci est la",__FILE__).' '.$first.' '.__("partie",__FILE__)

[Noyax37]

@nebz

ah par contre j’ai juste ajouté un espace en fin de chaine pour tester et je vois que pas mis dans les langues, tu fais un trim de ton coté ou c’est deepl ?

@Noyax37

Juste un petit conseil pour ceux qui suivront (s’ils le lisent), j’avais beaucoup de texte à traduire présentés comme ça

{{ blablabla }}

Le texte à traduire était bien " blablabla " mais le texte traduit était " blublublu", le dernier espace était supprimé. En soit cela ne pose pas de problème sauf si vous avez du texte qui arrive derrière et pour lequel il faut laisser un espace sinon les mots sont collés. J’ai donc mis les espaces en dehors des accolades et là il n’y a plus de problème.

c’est comme ça que j’ai résolu ce pb pour les {{}} et pour les autres je rajoute l’espace après avec . ' ' .