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

Passage à NodeJS 16 LTS (#6229) #6235

Merged
merged 3 commits into from
Feb 5, 2022

Conversation

SpaceFox
Copy link
Contributor

@SpaceFox SpaceFox commented Jan 23, 2022

Passage à NodeJS 16 LTS

D'après le site web de NodeJS, NodeJS 12 LTS va passer de "en maintenance" à "non supporté" le 30 avril 2022. Je propose de passer à NodeJS 16 LTS qui est en "actif" depuis le 26 octobre 2021, histoire de rester à jour.

QA :

  • nvm install puis make install-front
  • Vérifier les commandes make build-front, make watch-front, make lint-front
  • make run-back puis vérifier que site web s'affiche bien
  • Si les icônes ne s'affichent pas correctement c'est normal, il suffit de vider le cache du navigateur (Ctrl + F5 sur Firefox)

(Oui, j’ai copié/collé le message de MR de la précédente !)

@coveralls
Copy link

coveralls commented Jan 23, 2022

Coverage Status

Coverage remained the same at 87.025% when pulling 5bc0bbe on SpaceFox:fix-6229_node_16 into c6767d8 on zestedesavoir:dev.

@Arnaud-D Arnaud-D linked an issue Jan 23, 2022 that may be closed by this pull request
@philippemilink
Copy link
Member

Rapport de QA

Ça fonctionne, mais j'ai du faire make install-linux pour que nvm soit mis à jour, sans ça il ne connaissait pas la version 16 de node.

Ça demande un petit changement dans nos scripts Ansible : je mergerai la PR une fois que j'aurais fait et validé le changement.

@SpaceFox
Copy link
Contributor Author

SpaceFox commented Feb 3, 2022

Ha oui, forcément sur une install neuve je ne risquais pas de voir ce manque :D

@philippemilink
Copy link
Member

Ça demande un petit changement dans nos scripts Ansible : je mergerai la PR une fois que j'aurais fait et validé le changement.

Note à moi-même : le changement est minime dans les scripts Ansible et nécessite de mettre à jour manuellement (pas avec Ansible, mais avec apt update manuel) le paquet nodejs.

diff --git a/roles/common/tasks/nodejs.yml b/roles/common/tasks/nodejs.yml
index d26a580..f012c0d 100644
--- a/roles/common/tasks/nodejs.yml
+++ b/roles/common/tasks/nodejs.yml
@@ -6,7 +6,7 @@
 - name: add Node.js repository
   ansible.builtin.apt_repository:
     filename: nodejs
-    repo: "deb https://deb.nodesource.com/node_12.x {{ ansible_distribution_release }} main"
+    repo: "deb https://deb.nodesource.com/node_16.x {{ ansible_distribution_release }} main"
     state: present
 
 - name: add Node.js repository preferences

Pour l'instant (jusqu'au 30 avril) ce n'est pas bloquant puisqu'on n'utilise rien de spécifique à la version 16, il faudra s'en occuper au moment de la mise en production de la prochaine version de ZdS qui intégrera ce commit ou lors de la mise à jour des serveurs vers Debian 11 (on verra ce qui arrivera en premier).

@philippemilink philippemilink enabled auto-merge (squash) February 5, 2022 18:05
@philippemilink philippemilink merged commit 155e700 into zestedesavoir:dev Feb 5, 2022
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
Archived in project
Development

Successfully merging this pull request may close these issues.

Passage à Node.js v14 ou v16
3 participants