hook at glance #78
hook at glance #78
Conversation
sheplu
added
the
Pending Review
|
Deploy preview for fr-reactjs ready! Built with commit a487366 |
|
Deploy preview for fr-reactjs ready! Built with commit 5909ef3 |
fix mistakes
There was a problem hiding this comment.
Salut @sheplu !
Avant toute chose, merci pour tout ce boulot ! C'est super cool de nous filer un coup de main.
Y’a de belles choses là-dedans, mais ça sent la fatigue 😉, car y'a énormément de glissements sémantiques et de petites fautes. Entre-temps, le glossaire s'est un peu étoffé aussi, d'où une certaine normalisation, et des PRs sur les Hooks ont été finalisées, ce qui permet une certaine harmonisation.
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é. (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
added
Reviewed
and removed
Pending Review
Co-Authored-By: sheplu <sheplu@users.noreply.github.com>
Co-Authored-By: sheplu <sheplu@users.noreply.github.com>
|
je viens de finir de relire ta review, effectivement des simplifications qui donnent des phrases plus claires Par contre, j'ai validé et appliquer les modifs sur En tout cas, un grand merci pour la relecture @tdd ! |
|
@sheplu le mot "appli" n'existe pas plus que "démo", et pourtant ils sont en usage courant, y compris à l'écrit, dans des documents officiels, etc. En pratique, la VO utilise parfois “application” (qu'on traduira tel quel), parfois “app” (qu'on traduira en “appli”), pour préserver le rythme et le ton de la VO. Merci de ton traitement de la PR, je merge ça ASAP 👍 |
traduction de hook at glance
Je n ai pas trouve mieux que Apercu des Hooks en tant que titre, si vous avez mieux je suis preneur