zds-site
zds-site copied to clipboard
Les erreurs du linter front ne font pas échouer la CI
Remarqué sur la PR #6276 :
Le linter front trouve une erreur, mais le job GitHub Actions reste vert, bien qu'un message d'erreur soit affiché :
Il s'avère que Gulp, qui est lancé par make lint-front
et qui lance ensuite eslint, ne remonte pas le code de retour de eslint :
$ make lint-front
... erreur rapportée ...
$ echo $?
0
$ ./node_modules/eslint/bin/eslint.js assets/js/*.js
... erreur rapportée ...
$ echo $?
1
Il faut trouver comment faire pour que Gulp remonte le code d'erreur d'eslint, ou, à défaut, que la commande make lint-front
se charge d'analyser la sortie et produise le code de retour correspondant.
La documentation de la classe ESLint indique comment faire remonter le code d'erreur. Ce qu'il faut trouver c'est comment adapter le code pour que ça fonctionne avec Gulp.