-
Notifications
You must be signed in to change notification settings - Fork 0
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
[Refacto] Refactoring app with Vitejs React-ts and MUI #3
Conversation
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.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
☕ 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.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
💡 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é.
4a75272
to
b1dde0f
Compare
Playground for increase my skills (@LedruRomane) in ViteJs, React, Typescript and MUI / MUI integration.
[V1]