Skip to content
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

Traduction de la page "Composition vs Inheritance" #21

Merged
merged 18 commits into from
Feb 16, 2019

Conversation

LucasPerso
Copy link
Contributor

Bonjour à tous 👋
Voici ma traduction de la page Composition vs Inheritance

J'ai un peu hésité sur la traduction du titre en Composition contre Héritage ou Composition et Héritage. J'ai préféré le second choix qui me semblait plus harmonieux. C'est assez subjectif, je veux donc bien votre avis.

@LucasPerso LucasPerso changed the title Traduction de la page "Composition vs Héritage" Traduction de la page "Composition vs Heritage" Feb 13, 2019
@LucasPerso LucasPerso changed the title Traduction de la page "Composition vs Heritage" Traduction de la page "Composition vs Inheritance" Feb 13, 2019
@tdd tdd added the Pending Review Un mainteneur qualifié doit encore faire la revue de cette PR label Feb 13, 2019
@tdd tdd self-requested a review February 13, 2019 14:10
@tdd
Copy link
Collaborator

tdd commented Feb 13, 2019

@LucasPerso je regarde ça demain, j'ai trop de taf « normal » en retard là 😉

@tdd tdd self-assigned this Feb 13, 2019
Copy link

@exced exced left a comment

Choose a reason for hiding this comment

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

Hi, I hope you don't mind me reaching out of the blue here, I spotted some spelling mistakes

Copy link
Collaborator

@tdd tdd left a comment

Choose a reason for hiding this comment

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

Salut @LucasPerso !

Avant toute chose, merci pour tout ce boulot ! C'est super cool de nous filer un coup de main. Et d'ailleurs, pour une première PR c'est franchement pas mal.

Les principaux axes d'amélioration sont :

  • meilleure respiration des phrases (virgules judicieusement placées)
  • Arrêter avec « cela » 😉
  • Un peu de mot-à-mot sur les tournures de phrases, ne pas hésiter à bien saisir l'intention pour reformuler de façon plus naturelle, fluide, ou avec les infos de contexte clé suffisamment vers le début si la traduction étire trop le texte.

Tout est fait sous forme de Suggestions GitHub, il te suffit à chaque suggestion que tu approuves de cliquer le bouton d'ajout au lot de modifs à committer, et sur la fin de créer le commit groupé.

Si certaines appellent discussion, n'hésite pas, tu réponds, on en parle, on avance ensemble 🤝

Quand tout sera bouclé côté code, je ferai une repasse visuelle finale en local avant de fusionner.

Encore mille mercis, j'ai hâte de voir cette PR finalisée et dans master !

content/docs/composition-vs-inheritance.md Outdated Show resolved Hide resolved
content/docs/composition-vs-inheritance.md Outdated Show resolved Hide resolved
content/docs/composition-vs-inheritance.md Outdated Show resolved Hide resolved
content/docs/composition-vs-inheritance.md Outdated Show resolved Hide resolved
content/docs/composition-vs-inheritance.md Outdated Show resolved Hide resolved
content/docs/composition-vs-inheritance.md Show resolved Hide resolved
content/docs/nav.yml Outdated Show resolved Hide resolved
content/docs/composition-vs-inheritance.md Outdated Show resolved Hide resolved
content/docs/composition-vs-inheritance.md Outdated Show resolved Hide resolved
content/docs/composition-vs-inheritance.md Show resolved Hide resolved
@tdd tdd added Reviewed and removed Pending Review Un mainteneur qualifié doit encore faire la revue de cette PR labels Feb 14, 2019
tdd and others added 8 commits February 14, 2019 10:25
Co-Authored-By: LucasPerso <lucas.personnaz@gmail.com>
Co-Authored-By: LucasPerso <lucas.personnaz@gmail.com>
Co-Authored-By: LucasPerso <lucas.personnaz@gmail.com>
Co-Authored-By: LucasPerso <lucas.personnaz@gmail.com>
Co-Authored-By: LucasPerso <lucas.personnaz@gmail.com>
Co-Authored-By: LucasPerso <lucas.personnaz@gmail.com>
Co-Authored-By: LucasPerso <lucas.personnaz@gmail.com>
Co-Authored-By: LucasPerso <lucas.personnaz@gmail.com>
@LucasPerso
Copy link
Contributor Author

Merci @tdd et @exced pour vos retours pertinents !

Co-Authored-By: LucasPerso <lucas.personnaz@gmail.com>
@LucasPerso
Copy link
Contributor Author

@tdd Si c'est bon pour toi, c'est bon pour moi :)

@tdd
Copy link
Collaborator

tdd commented Feb 15, 2019

@LucasPerso il reste 8 suggestions que tu n'as pas traitées, et comme j'ai pas les droits d'écriture sur ta branche de fork (tu n'a pas coché "allow edit for maintainers"), je peux pas les passer…

tdd and others added 4 commits February 16, 2019 01:23
Co-Authored-By: LucasPerso <lucas.personnaz@gmail.com>
Co-Authored-By: LucasPerso <lucas.personnaz@gmail.com>
Co-Authored-By: LucasPerso <lucas.personnaz@gmail.com>
Co-Authored-By: LucasPerso <lucas.personnaz@gmail.com>
tdd and others added 4 commits February 16, 2019 01:25
Co-Authored-By: LucasPerso <lucas.personnaz@gmail.com>
Co-Authored-By: LucasPerso <lucas.personnaz@gmail.com>
Co-Authored-By: LucasPerso <lucas.personnaz@gmail.com>
Co-Authored-By: LucasPerso <lucas.personnaz@gmail.com>
@LucasPerso
Copy link
Contributor Author

Satanée "hidden conversations" de Github qui m'ont dupées comme il se doit. Navré pour le délai par conséquent, cette fois-ci, ça devrait être bon @tdd

@tdd tdd merged commit 50f81ea into reactjs:master Feb 16, 2019
@LucasPerso LucasPerso deleted the composition-vs-inheritance branch February 16, 2019 17:23
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants