Skip to content
This repository has been archived by the owner on May 7, 2022. It is now read-only.

French Version #26

Merged
merged 11 commits into from
Feb 28, 2015
Merged

French Version #26

merged 11 commits into from
Feb 28, 2015

Conversation

cedced19
Copy link
Contributor

Translation of the Roadmap to French.

I'd like someone else from @iojs/iojs-fr to verify please :)

<html lang="en">
<head>
<title>io.js Roadmap</title>
<meta charset="utf-8">
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Bin déjà, si on pouvait dire « feuille de route », ça serait pas plus mal. Bien connu et tout aussi compris que « roadmap », non ? Ici et ailleurs dans le doc…

@yoannmoinet
Copy link
Contributor

@bnb : @tdd and I are on it 👍
It's not ready yet. We'll notify you once it's done.

@bnb
Copy link

bnb commented Feb 27, 2015

Okay, thanks @yoannmoinet.

<li>Nous ne supprimons pas les API JS.</li>
<li><a href="http://semver.org">Versionnage sémantique</a>
<ul>
<li>Tout ajoût sur l'API signifie une incrémentation <em>mineure</em> de version.</li>
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Heu "ajoût" n'existe pas, désolé @yoannmoinet je sais pas d'où tu l'as sorti…

Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Effectivement, j'ai eu une bulle de cerveau je crois 😄. Navrés.

@tdd
Copy link
Contributor

tdd commented Feb 27, 2015

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 »…

@yoannmoinet
Copy link
Contributor

Je suis d'accord, le workflow est très mauvais. Et cela semble frustrant 😉.

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.

@mikeal
Copy link
Contributor

mikeal commented Feb 28, 2015

is this ready to merge?

@tdd
Copy link
Contributor

tdd commented Feb 28, 2015

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.

@tdd
Copy link
Contributor

tdd commented Feb 28, 2015

@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.

@tdd
Copy link
Contributor

tdd commented Feb 28, 2015

@cedced19 je viens de voir ton add ; tu veux que je finalise, ou tu t'en occupes ?

@cedced19
Copy link
Contributor Author

@tdd et @yoannmoinet je vous ai ajouté dans mon dépôts.

@cedced19
Copy link
Contributor Author

Tu peux finaliser. On perdra moins de temps tous les deux.

@tdd
Copy link
Contributor

tdd commented Feb 28, 2015

OK je fais ça et je repushe.

@tdd
Copy link
Contributor

tdd commented Feb 28, 2015

@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.

@cedced19
Copy link
Contributor Author

Je pense que c'est bon.

@yoannmoinet
Copy link
Contributor

je vais relire un coup.

@yoannmoinet
Copy link
Contributor

It's ok for me.
Good work @cedced19 and @tdd.

@tdd
Copy link
Contributor

tdd commented Feb 28, 2015

Alright then. @mikeal this can be merged, approved all around!

@mikeal
Copy link
Contributor

mikeal commented Feb 28, 2015

Can you add a link to this translations on the main page next to japanese and hebrew :)

@yoannmoinet
Copy link
Contributor

Ok, I'm updating the fork and adding the link.

Edit: Well I think a rebase might not be my brightest idea... (fixed it)

@yoannmoinet
Copy link
Contributor

Everything is fine @mikeal, I've added the link to the front-page too.

mikeal added a commit that referenced this pull request Feb 28, 2015
@mikeal mikeal merged commit 5b814c5 into nodejs:gh-pages Feb 28, 2015
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

6 participants