Translated the "How to contribute" page #93
Conversation
|
Deploy preview for fr-reactjs ready! Built with commit 588dad1 |
tdd
added
the
Pending Review
tdd
added
Reviewing
There was a problem hiding this comment.
Salut @adrienthiery !
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)
Bon bah écoute, pour une première trad' c'est franchement pas mal ! 👍 En revanche tu as totalement flingué mon processus de revue en prenant des libertés avec le formatage du Markdown, notamment en introduisant des sauts de ligne supplémentaires et une barre de séparation. Ça flingue les repères de l'algo de diff de Git/GitHub, du coup si tu regardes ton diff tu verras que les VF sont rarement en face des VO, ça sautille dans tous les sens, c'est le cauchemar…
J'aurais dû le repérer d'abord et ajuster le fichier en amont de ma revue, je l'ai pas fait, tant pis pour moi, j'ai donc souffert 😐
Côté contenu, le fond est plutôt bien, c'est surtout de la fluidification et du cosmétique (ortho / typo / grammaire), avec quelques points de terminologie (« changement » à éviter, « réparer / réparation » aussi (on préfère « corriger / correctif »), ce genre de truc).
Bref c'est quand même super cool, donc merci ! Maintenant, à toi de vérifier mes retours et de les intégrer. Comme c'est ta première contrib, lis bien attentivement la bonne façon de regrouper les acceptations de suggestion au début et à la fin de ce message. Ne viens pas pourrir mes notifs et l'histo en prime 😉
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
added
Reviewed
and removed
Reviewing
|
@adrienthiery j'espère que tu auras un moment pour traiter mes retours d'ici ce soir, sinon je devrai les auto-valider demain afin de boucler le périmètre de trad global ! Tiens-moi au courant, OK ? 😗 |
|
@adrienthiery bon faute de retours je voulais auto-valider les suggestions, sauf que je n'ai pas les droits sur ta PR ou son dépôt. Vérifie que tu as coché la case “allow maintainers to edit” dans ta PR, et si c'est déjà le cas y'a plus relou caché, donc ajoute-moi aux collaborateurs de ton fork svp. Sinon je vais devoir tout retranscrire à la main, je vais mourir 😢 |
|
Salut @tdd , désolé du délai, je n'avais pas internet pour toute cette dernière semaine... Le "allow maintainers to edit" est coché pourtant... Je vais t'ajouter en collaborateur. Désolé du délai, je suis au Chili sur un wifi franchement pas top avec seulement mon téléphone. 😅 |
|
Je t'ai invité à collaborer ! |
Co-Authored-By: adrienthiery <[email protected]>
|
Dammit, j'en ai oublié un 😰 |
|
Pas grave applique-le séparément |
Co-Authored-By: adrienthiery <[email protected]>
Salut !
Voila ma première passe sur la traduction. Désolé pour le nom de commit etc. (Ce serait chouette de squah & merge cette PR pour avoir un message plus correct) je suis en voyage et ai du laisser mon ordinateur chez une amie, donc je fais tout sur mobile 😅
Ayant eu aussi une connexion limitée a internet ces derniers jours, donc je n'ai pas pu checker les PR précédentes non plus...
J'espère que c'est quand meme de bonne qualité 😀