Closed cedced19 closed 9 years ago
Woot ! Eh ben, c'était pas du luxe :wink:
J'ai fini ma relecture, c'est un boulot massif, mais quelques affaires à noter :
Ok merci c'est la première fois que je traduis une page en complet. Merci.
I think we can push it!
Pas encore, car il y a encore des changements qui n'ont pas été appliqués.
Tu dois regarder les commentaires qui sont sur des outdated diff
, que tes commits ont fait disparaître.
On n'est pas pressés.
Car il faudra refaire une lecture finale avant de pouvoir pousser.
Je pensais avoir tout vu pourtant ok je verrai demain.
@cedced19 Can you get someone from @iojs/iojs-fr to +1 this directly, themselves?
@bnb : @tdd and I are on it :+1: It's not ready yet. We'll notify you once it's done.
Okay, thanks @yoannmoinet.
Au sujet de la remarque de @yoannmoinet :
« Fais attention à tes commits en plein milieu de review. Tu as perdu plein de commentaires en faisant ça. Tu devrais attendre que l'on finisse la review au complet pour faire ensuite tes changements. »
En fait non : on "perd" un commentaire si on modifie le diff sous-jacent, et le commentaire initial reste dans la discussion, juste replié sur lui-même.
En revanche, tu as de ton côté fait beaucoup de commentaires qui soit dupliquaient les miens, soit les contredisaient, ce qui laisse à penser que tu les as faits sur une version « initiale » du diff, et là on a clairement une amélioration de workflow à trouver pour éviter ces syndromes « un pas en avant, un pas en arrière »…
Je suis d'accord, le workflow est très mauvais. Et cela semble frustrant :wink:.
Mais je me tenais à jour sur ses commits. Je rafraîchissais la page à chaque nouveau poussé.
Je me suis même retrouvé à devoir ré-écrire plusieurs de mes commentaires qui n'étaient plus là, mais qui n'avaient pas été appliqués non plus.
is this ready to merge?
Hey @mikeal I'm afraid not quite yet, a few line notes remain unaddressed. Let me go ahead and fix that and I'll ping you then.
@mikeal Ah, dammit, I forgot for a blink this wasn't in iojs-fr. Can't take over that PR or @cedced19's source repo. We'll have to wait for him, but he said he'd tackle this today.
@cedced19 je viens de voir ton add ; tu veux que je finalise, ou tu t'en occupes ?
@tdd et @yoannmoinet je vous ai ajouté dans mon dépôts.
Tu peux finaliser. On perdra moins de temps tous les deux.
OK je fais ça et je repushe.
@yoannmoinet @cedced19 dernière relecture, idéalement avec le HTML ouvert en vrai dans vos browsers, svp. Je trouve qu'on est nickel là. Si pas de retours d'ici ce soir, je pingerai Mikeal pour la fusion.
Je pense que c'est bon.
je vais relire un coup.
It's ok for me. Good work @cedced19 and @tdd.
Alright then. @mikeal this can be merged, approved all around!
Can you add a link to this translations on the main page next to japanese and hebrew :)
Ok, I'm updating the fork and adding the link.
Edit: Well I think a rebase
might not be my brightest idea... (fixed it)
Everything is fine @mikeal, I've added the link to the front-page too.
Translation of the Roadmap to French.
I'd like someone else from @iojs/iojs-fr to verify please :)