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

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

Open
philippemilink opened this issue Mar 29, 2022 · 1 comment
Open

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

philippemilink opened this issue Mar 29, 2022 · 1 comment
Labels
C-DevelopmentEnv Amélioration de l'environnement de dev S-BUG Corrige un problème

Comments

@philippemilink
Copy link
Member

philippemilink commented Mar 29, 2022

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 philippemilink added S-BUG Corrige un problème C-DevelopmentEnv Amélioration de l'environnement de dev labels Mar 29, 2022
@Situphen
Copy link
Member

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.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
C-DevelopmentEnv Amélioration de l'environnement de dev S-BUG Corrige un problème
Projects
Status: À traiter
Development

No branches or pull requests

2 participants