-
Notifications
You must be signed in to change notification settings - Fork 3
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
[TECH] Importer seulement les fonctions de lodash qui sont utilisées. #303
[TECH] Importer seulement les fonctions de lodash qui sont utilisées. #303
Conversation
I'm deploying this PR to these urls:
Please check it out! |
Merci pour la découverte, je ne savais pas que cette manière d'importer était nécessaire pour le tree shaking ! |
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.
C'est dommage de garder lodash (qui reste d'une taille conséquente même avec un import ciblé) juste pour une fonction.
Est-ce qu'on ne pourrait pas s'en débarasser ?
J'ai poussé un essai là 18a4890 |
Merci 🙏 J'ai installé
|
Bon on arrive "que" à 294k, que fait-on ? |
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.
Ok pour cette PR.
L'idée de supprimer complètement lodash me parait bien pour le site vitrine.
(on peut toujours merger celle là d'abord)
3648930
to
c3c0d29
Compare
🦄 Problème
Lors des imports, nous importons tout Lodash.
🤖 Solution
Importer que les fonctions que nous utilisons.
🌈 Remarques
On passe alors de 318Kb gzippé à 299Kb.
💯 Pour tester