-
Notifications
You must be signed in to change notification settings - Fork 1
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
chore: update #7
base: main
Are you sure you want to change the base?
Conversation
Quality Gate passedIssues Measures |
20 |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Pourquoi ne pas passer directement à 22 LTS ?
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
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 |
deps maintenance and remove storybook