Closed thbar closed 1 week ago
👍 Dependency issues cleared. Learn more about Socket for GitHub ↗︎
This PR previously contained dependency changes with security issues that have been resolved, removed, or ignored.
J'ai fabriqué un diff et je l'ai donné à manger à ChatGPT et Claude.AI. Ca vaut ce que ça vaut et le risque d'hallucination est important, mais je le mets en appui à cette review.
Prompt: Can you summarise the components which have changed (upgraded, removed, added) between our master branch and our upgrade branch? Here is the yarn.lock diff.
Ah oui la violence du diff…
Oui, c'est tout le souci avec le stack Javascript pour la partie build. On peut fortement réduire notre stack javascript quand on passera au DSFR, en passant de webpack à esbuild (qui a beaucoup moins de dépendances et qui est le standard actuel dans Phoenix).
Juste pour être sûr : les titres des commits, c’est bien les commandes utilisées ? Si oui, peut-être le mettre dans le descriptif de la PR + mettre à jour le
learning_track.md
pour expliquer au prochain comment faire.
C'est tout à fait ça. Ça marche je prépare tout ça, c'est tout à fait rentable.
Report too large to display inline
Ah oui la violence du diff…
@vdegove et tu vois du coup pourquoi je n'ai passé que les MAJ mineures 🤣
Hey @vdegove:
Si tu as besoin de quelque chose pour finaliser la review de la présente PR, fais-moi signe !
Je fais une passe de MAJ sans chercher à passer les MAJ majeures (= je ne vais que rattraper les mineures pour l'instant), pour aller chercher les majeures plus sereinement après.
Ça ne touche plus ou moins que des éléments du stack de déploiement et pas trop des éléments fonctionnels, pour dégrossir.
À ne pas merger, j'aimerais:
prochainement
semaine prochaine et voir si je n'ai rien cassé -> OK pas de casse à première vuevoir si je peux générer un vague diffc'est pas fou, voir https://github.com/etalab/transport-site/pull/4287/filesCommandes utilisées
Je vais faire une PR de doc avec les commandes recommandées (bonne idée de @vdegove), les voilà rapidement:
Avant
Après