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 Static type checking #79

Merged
merged 9 commits into from
Mar 18, 2019

Conversation

clairefritz
Copy link
Contributor

Voici ma proposition de traduction pour "Static type checking" !

Quelques doutes quant à la terminologie française étant donné que ma langue de travail est l'anglais.

@netlify
Copy link

netlify bot commented Mar 2, 2019

Deploy preview for fr-reactjs processing.

Building with commit 954b71b

https://app.netlify.com/sites/fr-reactjs/deploys/5c8f9a52ab7c69000c245ee8

Copy link
Collaborator

@sheplu sheplu left a comment

Choose a reason for hiding this comment

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

bonne trad ! quelques petites erreurs de frappes sur différents mots
j'ai suggéré des traductions alternatives sur certaines endroits, ce n'est pas forcement le mieux mais ca me semble moins lourds des fois

pour les corrections, il suffit de relire et de les accepter directement via l'interface. si jamais certaines ne te conviennent pas, n'hesite pas a remettre un message pour en discuter

content/docs/static-type-checking.md Outdated Show resolved Hide resolved
content/docs/static-type-checking.md Outdated Show resolved Hide resolved
content/docs/static-type-checking.md Outdated Show resolved Hide resolved
content/docs/static-type-checking.md Outdated Show resolved Hide resolved
content/docs/static-type-checking.md Outdated Show resolved Hide resolved
content/docs/static-type-checking.md Outdated Show resolved Hide resolved
content/docs/static-type-checking.md Outdated Show resolved Hide resolved
content/docs/static-type-checking.md Outdated Show resolved Hide resolved
content/docs/static-type-checking.md Outdated Show resolved Hide resolved
content/docs/static-type-checking.md Outdated Show resolved Hide resolved
sheplu and others added 2 commits March 2, 2019 14:52
Co-Authored-By: clairefritz <fritz.claire@gmail.com>
@sheplu sheplu added the Pending Review Un mainteneur qualifié doit encore faire la revue de cette PR label Mar 2, 2019
@tdd tdd self-assigned this Mar 8, 2019
@tdd tdd added Reviewing Un mainteneur qualifié est en train de composer la revue de code and removed Pending Review Un mainteneur qualifié doit encore faire la revue de cette PR labels Mar 13, 2019
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 @clairefritz !

Avant toute chose, merci pour tout ce boulot ! C'est super cool de nous filer un coup de main.

Écoute c'est franchement pas mal, surtout pour une première trad ! 🥇 On sent que tu as bien lu les MP et le Glossaire, ça fait plaisir 😎. Bien sûr, il reste pas mal de petits couacs, ne serait-ce qu'en raison de la taille de la page, mais ce sont souvent les mêmes raisons pour tout un tas de petits retours. Rien de super problématique.

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 (pas de commit direct !), et sur la fin de créer le commit groupé. (ce mode de regroupement en un seul commit est à favoriser absolument, mais tu devras aller dans l'onglet Files changed pour y accéder, il n'est pas utilisable dans l’onglet Conversation)

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/nav.yml Outdated Show resolved Hide resolved
content/docs/static-type-checking.md Outdated Show resolved Hide resolved
content/docs/static-type-checking.md Outdated Show resolved Hide resolved
content/docs/static-type-checking.md Outdated Show resolved Hide resolved
content/docs/static-type-checking.md Outdated Show resolved Hide resolved
content/docs/static-type-checking.md Outdated Show resolved Hide resolved
content/docs/static-type-checking.md Outdated Show resolved Hide resolved
content/docs/static-type-checking.md Outdated Show resolved Hide resolved
content/docs/static-type-checking.md Outdated Show resolved Hide resolved
content/docs/static-type-checking.md Outdated Show resolved Hide resolved
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 @clairefritz !

Avant toute chose, merci pour tout ce boulot ! C'est super cool de nous filer un coup de main.

Écoute c'est franchement pas mal, surtout pour une première trad ! 🥇 On sent que tu as bien lu les MP et le Glossaire, ça fait plaisir 😎. Bien sûr, il reste pas mal de petits couacs, ne serait-ce qu'en raison de la taille de la page, mais ce sont souvent les mêmes raisons pour tout un tas de petits retours. Rien de super problématique.

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 (pas de commit direct !), et sur la fin de créer le commit groupé. (ce mode de regroupement en un seul commit est à favoriser absolument, mais tu devras aller dans l'onglet Files changed pour y accéder, il n'est pas utilisable dans l’onglet Conversation)

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 !

@tdd tdd added Reviewed and removed Reviewing Un mainteneur qualifié est en train de composer la revue de code labels Mar 13, 2019
@tdd
Copy link
Collaborator

tdd commented Mar 14, 2019

@clairefritz si tu peux trouver un moment d'ici demain après-midi pour traiter mes retours ce serait le top 😉

@tdd
Copy link
Collaborator

tdd commented Mar 16, 2019

@clairefritz pourrais-tu trouver le temps de traiter mes retours d'ici lundi midi ? pour info, si ce n'est pas le cas, j'auto-validerai les suggestions pour avancer.

Co-Authored-By: clairefritz <fritz.claire@gmail.com>
@clairefritz
Copy link
Contributor Author

@tdd Done 🚀

Co-Authored-By: clairefritz <fritz.claire@gmail.com>
@tdd
Copy link
Collaborator

tdd commented Mar 18, 2019

@clairefritz cool je fais ma dernière repasse visuelle en local et on y va 😉

@clairefritz
Copy link
Contributor Author

🎉

Unbreakable spaces, orphan/widow management, etc.
@tdd tdd merged commit a6a5016 into reactjs:master Mar 18, 2019
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