chore: update #7
Conversation
|
| 20 |
There was a problem hiding this comment.
Pourquoi ne pas passer directement à 22 LTS ?
There was a problem hiding this comment.
ha j'avais pas vu ta PR #6
|
C'est pas grand chose mais ça pourrait être bien de différencier les pages destinées à rester (404.tsx) de celles à virer (mui.tsx), par exemple en les mettant dans un sous-répertoire de pages ? Et peut-être en profiter pour vider |
|
oui, et de migrer sur app-router ? on peut virer le BASE_PATH oui, on en a juste besoin pour la demo sur GitHub |
|
Je ne suis pas assez expert pour me prononcer sur app vs page router ; mais dans le cas d'un template on peut éventuellement laisser les deux (il me semble qu'ils peuvent coexister) et laisser les utilisateurs choisir ? |
|
Une autre proposition serait de virer dsfr-chart ; dans mon projet c'est une des premières choses que j'ai faite et ça a beaucoup accéléré le |
l'intégration a peut-être évolué depuis, je peux regarder dans une autre PR |
|
Il faudrait aussi updater la version de node dans https://github.com/betagouv/template-nextjs/blob/main/.github/workflows/lint.yml , dans le package.json (engines + @types/node) et dans le Dockerfile? Je n'ai pas eu de problèmes avec 22.x je propose qu'on future-proof en adoptant celle là partout :-) |
|
Idéalement il faudrait aussi bouger mdx-components dans src/ ? |
|
go itérer sur la PR si tu as un peu de temps :) |
deps maintenance and remove storybook