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 @bastiendmt !
Avant toute chose, merci pour tout ce boulot ! C'est super cool de nous filer un coup de main.
En revanche sur cette PR précise, écoute, je vais être honnête… c’est pas quali du tout 😳 Tu sais faire beaucoup mieux que ça, mais j'ai la nette impression que tu as "torché" cette contribution sans relecture ni chercher à faire des phrases fluides. Du coup ça m'a sans doute pris plus de temps de faire la revue que de traduire moi-même.
Je comprends ton enthousiasme à participer (et c'est top !), mais ne confonds pas vitesse et précipitation la prochaine fois, d'accord ? 😉
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 !
|
Hello @tdd désolé pour toutes les fautes, je reconnais être allé plus vite que la normale sur cette PR. Je ferai mieux la prochaine fois. |
Co-authored-by: Christophe Porteneuve <tdd@tddsworld.com>
Hello, voici ma PR pour la page
Profiler😁