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 @linsolas !
Avant toute chose, merci pour tout ce boulot ! C'est super cool de continuer à nous filer un coup de main.
Comme d'hab' c'est très bien 😎 même si j'ai repéré quelques couacs que tu ne fais pas d'habitude (et sauvé un paragraphe de l'oubli 😁).
Et pour rappel : pas de ponctuation en fin de liens !
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 @tdd pour la relecture, et encore désolé pour le pauvre paragraphe supprimé, qui n'avait rien fait !! J'ai laissé tes corrections concernant pour les |
|
Done |
Co-authored-by: Christophe Porteneuve <tdd@tddsworld.com>
Co-authored-by: Christophe Porteneuve <tdd@tddsworld.com>
Co-authored-by: Christophe Porteneuve <tdd@tddsworld.com>
Traduction de la page
<StrictMode>