Conversation
Size changesDetails📦 Next.js Bundle Analysis for react-devThis analysis was generated by the Next.js Bundle Analysis action. 🤖 This PR introduced no changes to the JavaScript bundle! 🙌 |
tdd
left a comment
There was a problem hiding this comment.
Salut @paulus85 !
Avant toute chose, merci pour tout ce boulot ! C'est super cool de nous filer un coup de main.
Pour l'essentiel ce sont des correctifs de conventions typographiques (insécables devant les deux-points, pas de majuscule après, pas de ponctuation en fin de lien, etc.), ainsi que quelques reformulations, notamment lorsque c'était un peu trop mot-à-mot.
Mais sinon cool ! 👍
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 🤝
Quand tout sera bouclé côté code, je ferai une repasse visuelle finale en local avant de fusionner. J'y recalerai si besoin les usual suspects : espaces insécables avant les ponctuations doubles, et apostrophes françaises.
Encore mille mercis, j'ai hâte de voir cette PR finalisée et dans main !
|
Merci pour ton retour @tdd ! |
Co-authored-by: Christophe Porteneuve <tdd@tddsworld.com>
No description provided.