Skip to content
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] Migrer de nuxt-fontawesome à @nuxtjs/fontawesome #447

Merged
merged 1 commit into from
Dec 22, 2022

Conversation

yannbertrand
Copy link
Member

@yannbertrand yannbertrand commented Dec 20, 2022

🦄 Problème

On utilise la dépendance nuxt-fontawesome pour intégrer nos icônes fontawesome. Il est recommandé de migrer vers la version de l'organisation officielle nuxt : @nuxtjs/fontawesome.

🤖 Solution

Utiliser la nouvelle dépendance @nuxtjs/fontawesome.

🌈 Remarques

@nuxtjs/fontawesome est maintenant un buildModule et plus un module.

Je ne suis pas sûr de si cela doit être une dependency ou une devDependency, j'ai gardé ce que l'on avait jusqu'à présent.

💯 Pour tester

Vérifier que les icônes s'affichent toujours.

@pix-service
Copy link

I'm deploying this PR to these urls:

Please check it out!

nuxt.config.js Outdated Show resolved Hide resolved
"version": "2.0.6",
"resolved": "https://registry.npmjs.org/@fortawesome/vue-fontawesome/-/vue-fontawesome-2.0.6.tgz",
"integrity": "sha512-V3vT3flY15AKbUS31aZOP12awQI3aAzkr2B1KnqcHLmwrmy51DW3pwyBczKdypV8QxBZ8U68Hl2XxK2nudTxpg==",
"version": "0.1.10",
Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

😢

"@fortawesome/vue-fontawesome": "^0.1.4"
},
"dependencies": {
"@fortawesome/vue-fontawesome": {
Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Comme l'impression qu'on avait ajouter @fortawesome/vue-fontawesome pour être à jour car le problème était déjà là

Copy link
Member Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

En fait en ajoutant @fortawesome/vue-fontawesome en v2, ça ajoute juste une dépendance mais utilise toujours la 0.1.10 quand demandé par @nuxtjs/fontawesome...

C'était déjà le cas avant, on a :

  • fortawesome/vue-fontawesome en 2.0.6 que l'on demande dans nos dependencies
  • nuxt-fontawesome 0.4.0 qui demande @fortawesome/vue-fontawesome en 0.1.10 (on l'a aussi dans le package-lock)

Du coup je sais pas si ça vaut le coup de rajouter cette dep supplémentaire ?

Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Il est vrai qu'on faisait pas d'override sur la configuration, mais en même temps, je ne vois pas de ligne qui "supprime" une des 2 versions, on peut voir qu'un remplacement simple ligne : 24759 🤔

Copy link
Member Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

En réalité, je pense qu'il se servait de celui récupéré ligne 14698 (node_modules/nuxt-fontawesome/node_modules/@fortawesome/vue-fontawesome)

Copy link
Member Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Mais on est très profond dans la mécanique interne de npm là ^^"

Copy link
Member Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

En tout cas je ne pense pas que rajouter une dépendance de notre côté soit suffisant pour qu'une autre dépendance en bénéficie. Je crois qu'il y a des mécaniques pour surcharger volontairement des versions de dépendances de dépendances mais je sais pas si on a intérêt à le faire...

@yannbertrand yannbertrand force-pushed the tech-use-nuxtjs-fontawesome branch from 3c15d30 to 717ac7f Compare December 20, 2022 13:56
@yannbertrand yannbertrand force-pushed the tech-use-nuxtjs-fontawesome branch from 717ac7f to fdb9ded Compare December 20, 2022 14:13
@yannbertrand yannbertrand force-pushed the tech-use-nuxtjs-fontawesome branch from fdb9ded to 90c5359 Compare December 22, 2022 10:19
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

5 participants