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

MIse à jour de Node.js #1642

Merged
merged 1 commit into from
Feb 20, 2025
Merged

MIse à jour de Node.js #1642

merged 1 commit into from
Feb 20, 2025

Conversation

Mopolo
Copy link
Contributor

@Mopolo Mopolo commented Feb 19, 2025

Important

Il faudra mettre à jour la variable d'env CC_NODE_VERSION sur Clever Cloud avant de deploy cette PR

La lib node-sass est dépréciée depuis longtemps (et ne marche pas bien sur les processeurs Apple, même via Docker) et la nouvelle lib a la même API : https://sass-lang.com/blog/libsass-is-deprecated/

Yarn n'a plus autant d'intérêt aujourd'hui. NPM va tout aussi vite, a un lockfile par défaut depuis un moment et est installé avec Node de toute façon donc on évite une dépendance de plus.

Le Dockerfile contient à nouveau une version spécifique de Node pour éviter les mises à jour surprise quand l'image Debian de base change de version (actuellement la version de base est la v12 alors qu'on est en v6 sur le serveur de prod). La version 22 est l'actuelle LTS.

Infos : https://nodejs.org/en/about/previous-releases

Et sur l'interface de Clever Cloud :
image

La lib `node-sass` est dépréciée depuis longtemps et la nouvelle lib a
la même API : https://sass-lang.com/blog/libsass-is-deprecated/

Yarn n'a plus autant d'intérêt aujourd'hui. NPM va tout aussi vite, a
un lockfile par défaut depuis un moment et est installé avec Node de
toute façon donc on évite une dépendance de plus.

Le Dockerfile contient à nouveau une version spécifique de Node pour
éviter les mises à jour surprise quand l'image Debian de base change de
version (actuellement la version de base est la v12 alors qu'on est en
v6 sur le serveur de prod). La version 22 est l'actuelle LTS.

Infos : https://nodejs.org/en/about/previous-releases
@Mopolo Mopolo self-assigned this Feb 19, 2025
@Mopolo Mopolo marked this pull request as ready for review February 19, 2025 22:13
Copy link
Contributor

@stakovicz stakovicz left a comment

Choose a reason for hiding this comment

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

Trop bien !
Ça veut dire qu'on va pouvoir enlever la platform linux/amd64 pour me puce Apple ??
😍 😍

@Mopolo
Copy link
Contributor Author

Mopolo commented Feb 20, 2025

Ça veut dire qu'on va pouvoir enlever la platform linux/amd64 pour me puce Apple ??

Hélas nan pas encore 😅
La version de MySQL qu'on utilise en a encore besoin.

Mais au moins ce sera plus facile de compiler les assets depuis un Mac

@stakovicz
Copy link
Contributor

Ha oui ... MySQL, c'est la next step ;-)
Ce midi, je cherry-pick ton commit sur la preprod pour valider le changement de la version de Node.JS et si c'est bon, je MEP dans la foulée.

@stakovicz stakovicz merged commit 4c13237 into afup:master Feb 20, 2025
4 checks passed
@Mopolo Mopolo deleted the update-node-and-npm branch February 20, 2025 12:52
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.

3 participants