zds-site icon indicating copy to clipboard operation
zds-site copied to clipboard

Les erreurs du linter front ne font pas échouer la CI

Open philippemilink opened this issue 2 years ago • 1 comments

Remarqué sur la PR #6276 : image

Le linter front trouve une erreur, mais le job GitHub Actions reste vert, bien qu'un message d'erreur soit affiché :

image

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.

philippemilink avatar Mar 29 '22 21:03 philippemilink

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.

Situphen avatar Mar 29 '22 21:03 Situphen