[Refacto] Refactoring app with Vitejs React-ts and MUI #3
Conversation
LedruRomane
force-pushed
the
refacto-vite-react-ts
branch
from
40a8b0f
to
b6fe3bb
Compare
|
|
||
| // React | ||
| // https://eslint.org/docs/latest/rules/jsx-quotes | ||
| 'jsx-quotes': ['error', 'prefer-double'], |
There was a problem hiding this comment.
☕ later: A traiter dans une autre PR plus tard éventuellement.
Eslint a décidé de ne plus supporter nativement les rules concernant le formattage de code, pour se concentrer davantage sur la qualité de code que le CS. Ils recommandent d'utiliser des outils tels que Prettier mais qui sont extrêmement opinionated et peu configurable. Or le formatting offert par Prettier génère beaucoup trop de différentiel lors de certains changements de code, et ne favorise pas toujours la lisibilité (pas de multiline avant d'atteindre les 120 chars par exemple). No-go pour moi.
Heureusement, la communauté maintient désormais les rules eslint de formatting dans des packages dédiés.
Cf https://github.com/Elao/amabla/pull/180 pour un exemple de migration vers ces nouvelles règles (basiquement, juste à les préfixer et installer les nouveaux packages et plugins)
| @@ -1,53 +1,30 @@ | |||
| # Ça-mâche-quoi | |||
There was a problem hiding this comment.
💡 suggestion: Ré-introduire un README.md propre au contenu du projet et supprimer/déplacer le contenu de celui généré par Vite dans un espace/fichier dédié.
LedruRomane
force-pushed
the
refacto-vite-react-ts
branch
from
4a75272
to
b1dde0f
Compare
Playground for increase my skills (@LedruRomane) in ViteJs, React, Typescript and MUI / MUI integration.
[V1]