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.

Sign up for GitHub

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

Jump to bottom

Mise à jour des dépendances front #6261

Merged
merged 4 commits into from
Mar 25, 2022
Merged

Mise à jour des dépendances front #6261

merged 4 commits into from
Mar 25, 2022

Conversation

Situphen
Copy link
Member

@Situphen Situphen commented Mar 7, 2022 •
edited
Loading

Mise à jour des dépendances front, sauf gulp-imagemin car c'est maintenant un ESM (EcmaScript Module) donc il faut convertir Gulpfile.js en ESM lui aussi pour que ça fonctionne (ce que je n'ai pas réussi).

QA :

  • source zdsenv/bin/activate && make update
  • Vérifier que les commandes make build-front, make watch-front, make format-front, make lint-front et make clean-front fonctionnent correctement
  • Relire les modifications effectuées dans le code JS, notamment le commit « Correction manuelle des erreurs ESLint »
  • Vérifier le bon fonctionnement du JS sur le site web, notamment les fonctionnalités dont le code a été modifié par le commit « Correction manuelle des erreurs ESLint »

@Situphen Situphen added C-Front Concerne l'interface du site C-DevelopmentEnv Amélioration de l'environnement de dev labels Mar 7, 2022
@Situphen Situphen force-pushed the nodejs-deps branch 2 times, most recently from b703ac5 to ebf8005 Compare March 7, 2022 16:27
@Situphen Situphen marked this pull request as ready for review March 7, 2022 16:42
@Situphen Situphen added this to En développement in Suivi des PR via automation Mar 7, 2022
@Situphen Situphen moved this from En développement to En attente de QA in Suivi des PR Mar 7, 2022
@coveralls
Copy link

coveralls commented Mar 7, 2022 •
edited
Loading

Coverage Status

Coverage remained the same at 87.32% when pulling c7da62c on Situphen:nodejs-deps into f67a5e6 on zestedesavoir:dev.

Suivi des PR automation moved this from En attente de QA to Fusionnable après rebase Mar 25, 2022
philippemilink
philippemilink approved these changes Mar 25, 2022
View reviewed changes
Copy link
Member

@philippemilink philippemilink left a comment

Choose a reason for hiding this comment

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

Rapport de QA

OK ✔️

@philippemilink philippemilink merged commit 1a3bc43 into zestedesavoir:dev Mar 25, 2022
Suivi des PR automation moved this from Fusionnable après rebase to Fusionnée Mar 25, 2022
@Situphen Situphen deleted the nodejs-deps branch March 25, 2022 18:48
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@philippemilink philippemilink philippemilink approved these changes

Assignees
No one assigned
Labels
C-DevelopmentEnv Amélioration de l'environnement de dev C-Front Concerne l'interface du site
Projects
Suivi des PR
Archived in project
Suivi des PR
  
Fusionnée
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.

None yet
3 participants
@Situphen @coveralls @philippemilink