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

🌐 Translation blog post 2017-09-08 #245

Merged
merged 2 commits into from Jan 31, 2020

Conversation

lbelavoir
Copy link
Contributor

@lbelavoir lbelavoir commented Jan 30, 2020

@tdd dĂ©solĂ©e du retard, cette traduction n'Ă©tait complĂštement sortie de la tĂȘte ! EspĂ©rant que tu aies pu rattraper ton retard de relecture.

@netlify
Copy link

netlify bot commented Jan 30, 2020

Deploy preview for fr-reactjs ready!

Built with commit c8d8b0c

https://deploy-preview-245--fr-reactjs.netlify.com

@lbelavoir lbelavoir requested a review from tdd January 30, 2020 04:25
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 @lbelavoir !

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

Avant de traiter les retours, lis bien TOUT ce message, notamment les consignes d'intégration optimale des retours par lot(s)

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 đŸ€

La majorité des retours tient à quelques points récurrents :

  • Espaces insĂ©cables avant les signes de ponctuation doubles
  • On ne traduit jamais les ancres de titres
  • On Ă©vite « cela », pour utiliser soit « ça » soit un qualificateur explicite
  • Y’avait pas mal de mot-Ă -mot

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/blog/2017-09-08-dom-attributes-in-react-16.md Outdated Show resolved Hide resolved
content/blog/2017-09-08-dom-attributes-in-react-16.md Outdated Show resolved Hide resolved
content/blog/2017-09-08-dom-attributes-in-react-16.md Outdated Show resolved Hide resolved
content/blog/2017-09-08-dom-attributes-in-react-16.md Outdated Show resolved Hide resolved
content/blog/2017-09-08-dom-attributes-in-react-16.md Outdated Show resolved Hide resolved
content/blog/2017-09-08-dom-attributes-in-react-16.md Outdated Show resolved Hide resolved
content/blog/2017-09-08-dom-attributes-in-react-16.md Outdated Show resolved Hide resolved
content/blog/2017-09-08-dom-attributes-in-react-16.md Outdated Show resolved Hide resolved
content/blog/2017-09-08-dom-attributes-in-react-16.md Outdated Show resolved Hide resolved
content/blog/2017-09-08-dom-attributes-in-react-16.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 @lbelavoir !

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

Avant de traiter les retours, lis bien TOUT ce message, notamment les consignes d'intégration optimale des retours par lot(s)

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 đŸ€

La majorité des retours tient à quelques points récurrents :

  • Espaces insĂ©cables avant les signes de ponctuation doubles
  • On ne traduit jamais les ancres de titres
  • On Ă©vite « cela », pour utiliser soit « ça » soit un qualificateur explicite
  • Y’avait pas mal de mot-Ă -mot

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 the Reviewed label Jan 30, 2020
Co-Authored-By: Christophe Porteneuve <tdd@tddsworld.com>
@tdd tdd merged commit 757b951 into reactjs:master Jan 31, 2020
@lbelavoir lbelavoir deleted the 2017-09-08-dom-attributes-in-react-16 branch January 31, 2020 15:37
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.

None yet

2 participants