nodejs / nodejs-fr

Node.js en français.
https://nodejs.org
Other
30 stars 14 forks source link

Iojs week 2015 03 20 #48

Closed yoannmoinet closed 9 years ago

yoannmoinet commented 9 years ago

Voici la traduction qui fixe #45.

Pour avoir une meilleure vue, vous pouvez regarder seulement le deuxième comit.

Le premier est simplement l'ajout du fichier.

J'ai marqué [??] là où je ne trouvais pas de traduction. À deux endroits :

Merci de revoir ça vite, on est déjà pas mal à la bourre :smile:.

podefr commented 9 years ago

Merci pour la traduction! Par contre, je suis complètement perdu. Le nom du PR est 2015 02 27, le nom du fichier est 2015 03 20, le titre est Week of March 13, alors qu'il semble bien qu'il s'agisse de la traduction de l'update du 20 mars. Pourrais-tu uniformiser tout ça, y compris le nom de la branche pour éviter toute confusion?

yoannmoinet commented 9 years ago

Merci pour ta revue @podefr, je vais corriger ça dans la soirée.

Désolé pour l’ambiguïté avec les noms :/ j'ai dû m'embrouiller dans mes copié-collés. Cependant, je ne crois pas pouvoir changer le nom de la branch sans devoir resoumettre une pull-request.

Est-ce ce que tu veux que je fasse?

Je changerais les autres noms.

podefr commented 9 years ago

Tant pis pour le renommage de la branche, peux-tu juste changer le titre du PR? merci!

yoannmoinet commented 9 years ago

J'ai appliqué tes corrections. Juste un dernier doute sur https://github.com/iojs/iojs-fr/pull/48#discussion_r27450004 comme énoncé dans mon commentaire en réponse.

podefr commented 9 years ago

On y est presque, reste à corriger le titre (20 mars au lieu de 13) Ensuite on pourra merger :)

Jimmy-Dinis commented 9 years ago

Est-ce que tout est ok ?

podefr commented 9 years ago

@steel-finger non le titre n'est toujours pas bon

yoannmoinet commented 9 years ago

J'ai fixé le titre.