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 triviale des dépendances Node.js #6230

Merged
merged 1 commit into from
Feb 5, 2022

Conversation

Situphen
Copy link
Member

Mise à jour triviale des dépendances Node.js car cela n'avait pas été fait depuis quelques temps déjà !

  • Je n'ai pas mis à jour les dépendances de développement car la mise à jour n'est pas triviale (enfin, ce n'était pas triviale lors de ma dernière PR mais je n'ai pas réessayé depuis ceci dit).
  • Je n'ai pas mis à jour gulp-imagemin puisqu'il est passé sous la forme d'un ESM et donc demande pas mal d'adaptation de notre côté j'ai l'impression.
  • D'une manière générale, pas mal de paquets ont l'air de ne plus trop recevoir de mises à jour. Je pense que cela s'explique notamment car Gulp est un peu passé de mode. Il faudrait réfléchir à migrer vers un autre écosystème un peu plus à la mode pour s'assurer d'avoir des dépendances à jour, je pense.

QA :

  • source zdsenv/bin/activate && make update && make zmd-start && make run-back
  • S'assurer que la partie front du site web fonctionne bien

@Situphen Situphen added the C-Front Concerne l'interface du site label Jan 22, 2022
@coveralls
Copy link

coveralls commented Jan 22, 2022

Coverage Status

Coverage remained the same at 87.025% when pulling 45adbd4 on Situphen:nodejs-deps into d3fc65f on zestedesavoir:dev.

@philippemilink philippemilink merged commit 7b389ba into zestedesavoir:dev Feb 5, 2022
@Situphen Situphen deleted the nodejs-deps branch February 5, 2022 21:21
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
C-Front Concerne l'interface du site
Projects
Archived in project
Development

Successfully merging this pull request may close these issues.

3 participants