-
Notifications
You must be signed in to change notification settings - Fork 266
Rectification des *fuzzy* du fichier sys.po
.
#1099
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Conversation
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Merci pour cette PR ! Quelques remarques.
Ah bon ? Tant mieux et merci encore.
|
Oui, car une « Pull Request » c'est litéralement une demande de merger une branche. Quand tu vas |
Ah super ! Je vais rectifier ce que me dit padpo et faire un nouveau
*commit*. Merci encore pour l'explication.
|
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Relu jusqu'a la ligne 156.
@louisMAURY Bonjour Louis. Lorsque tu te remet à traduire comme là deux jours plus tard sur un sujet légèrement différent (ici OK c'est discutable, c'est le même fichier, mais il y a deux jours tu travaillais sur des fuzzy, et là tu travaille sur des traductions), tu devrais créer une nouvelle branche, et donc une nouvelle PR, comme ça on peux avancer sur le merge du premier travail pendant que tu es sur le second. Pour faire une nouvelle branche, fie toi à CONTRIBUTING.rst qui dit en gros :
La première commande permet de s'assurer que tu es à jour, la 2ème fait partir ta nouvelle branche sur la version à jour de la 3.8. Ça ne marche que si chez toi la Pour ce coup ci, ne change rien, ne t'embête pas, mais pour le prochain coup fais une branche. |
C'est effectivement le cas, puisque la recommandation pour toute nouvelle contribution est le paragraphe que tu cites « en gros », et qu'on a évoqué. Par contre j'ai pas insisté sur le fait de faire deux branches pour deux modifications non corrélées dans un seul et même fichier… |
@vpoulailleau c'est pas très gênant, ça va retarder un peu le merge des fuzzy, mais y'a pas le feu au lac :) |
D'accord je vais faire attention. Et dans une correction je dois modifier
le mot *hooks* par "des fonctions d'appel" est-ce que j'applique ce
changement sur toute les fois où j'ai marqué *hooks* ?
Je suis vraiment désolé pour tout ce bazar...
Le lun. 16 déc. 2019 à 15:38, Julien Palard <[email protected]> a
écrit :
… @vpoulailleau <https://github.com/vpoulailleau> c'est pas très gênant, ça
va retarder un peu le merge des fuzzy, mais y'a pas le feu au lac :)
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub
<#1099?email_source=notifications&email_token=AKLEESHDBT4NA2EQZV7FKV3QY6HHRA5CNFSM4J2ZZR52YY3PNVWWK3TUL52HS4DFVREXG43VMVBW63LNMVXHJKTDN5WW2ZLOORPWSZGOEG65HEA#issuecomment-566088592>,
or unsubscribe
<https://github.com/notifications/unsubscribe-auth/AKLEESHJMX3AKEMSPF62FFTQY6HHRANCNFSM4J2ZZR5Q>
.
|
Pour t'aider, tu peux regarder l'article wikipédia de hook. Plusieurs traductions sont possibles : fonction de rappel, point d'entrée, voire procédure automatique… Tu peux varier et choisir celui qui correspond le mieux au contexte. |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Quelques petits soucis d'orthographe et c'est bon pour moi.
@louisMAURY j'ai poussé ma relecture de décembre et j'ai pu merger ta branche, merci pour cette PR :) |
Fix #1089