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

chore: upgrade nuxt and node #127

Merged
merged 3 commits into from
Jan 31, 2025
Merged

Conversation

ghusse
Copy link
Contributor

@ghusse ghusse commented Dec 27, 2024

Mise à jour de nuxt avec la dernière version 2. Nécessite de passer à node 18 à cause des dépendances postCSS + nécessite de préciser des versions dans la section "overrides" car les dernières versions des packages en question posent problème.

Normalement, avec tout ça, la mise à jour est possible et ça fonctionne bien.

@lucien-chastan
Copy link
Contributor

Hello !
Merci pour cette update !
Je n'ai pas encore eu le temps de tester voir je ne n'ai pas d'effet de bord

J'ai vu qu'il y avait la version v2.18.1 qui était là
Tu as testé ? Il y avait un bloquant ?

(déjà l'update vers 2.17.4 c'est déjà un très bon pas en avant !)

@ghusse
Copy link
Contributor Author

ghusse commented Dec 28, 2024 via email

@lucien-chastan
Copy link
Contributor

2.17.4 c'est très bien ! Je sais que c'est la galère les montés de versions, tu sais quand tu commence mais jamais quand tu finira 🤣, c'est d'ailleurs pour ça que j'ai autant de retard sur les updates de certain package

C'est drôle, le temps de build du client et serveur est quasiment deux fois plus long

Nuxt 2.15.8 + node 16.14.0
image

Nuxt 2.17.4 + node 18.20.5
image

On pourrai s'attendre à l'inverse en montant de version
En soit ce n'est pas grave, j'attends une minute, je peux attendre une minute 50 😌

je me suis baladé sur l'app dans les endroits qui me semblait sensible, j'ai trouvé aucun bugs
Je vais mettre ce développement sur la version next et laisser quelques temps avant de passer en prod

@lucien-chastan
Copy link
Contributor

Je fais des tests avec 2.18.1, ça a l'air de bien fonctionner aussi !

@ghusse
Copy link
Contributor Author

ghusse commented Dec 28, 2024

Que donne le temps de build avec cette version ?

@ghusse
Copy link
Contributor Author

ghusse commented Dec 28, 2024

J'ai mis à jour cette PR avec la version 2.18

@lucien-chastan
Copy link
Contributor

Super merci !
Le build est peut-être 10 secondes plus rapide qu'en 2.17.4, mais ça reste plus lent que 2.15.8
Mais ça dépend peut-être juste de la disponibilité de ma machine au moment du build, j'ai pas faite des itérations pour faire une moyen

@lucien-chastan
Copy link
Contributor

Hello !
Désolé du temps de traitement
J'ai eu quelques surprises avec node 18 et pm2 pour que j'ai du mettre à jour et qui "diffuse" (je ne suis pas sur du terme) sur ipv6, bref j'ai appris des choses
Ça a l'air de bien fonctionner ! Hormis le temps de build un peu plus long
Je pense passer cette PR en prod la semaine prochaine

Encore merci !

@ghusse
Copy link
Contributor Author

ghusse commented Jan 25, 2025 via email

@lucien-chastan lucien-chastan merged commit 06af105 into oblyk:master Jan 31, 2025
3 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants