Script de pre-commit plus robuste #623

Merged

mpepin merged 1 commit from Kerl/linters into aureplop/linters

2018-10-06 16:04:38 +02:00

mpepin commented

2018-10-06 16:01:32 +02:00

(Migrated from git.eleves.ens.fr)

Applique les changements suggérés par shellcheck sur les deux points suivants :

SC2181 : en l'occurrence, certaines erreurs étaient mal rattrapées car le printf "OK\n" met la variable $? à 0
SC2086 : ça n'a pas d'effet en l'état actuel des choses mais on prévient des bugs possible si jamais un nom de fichier contient un jour un caractère de type espace, \n, tabulation, etc.

Shellcheck trigger un autre warning sur ce fichier (SC1117). Je l'ignore délibérément, ce n'est pas vraiment une source de bug et il est sur le point d'être désactivé par shellcheck.

Applique les changements suggérés par shellcheck sur les deux points suivants : - [SC2181](https://github.com/koalaman/shellcheck/wiki/SC2181) : en l'occurrence, certaines erreurs étaient mal rattrapées car le `printf "OK\n"` met la variable `$?` à `0` - [SC2086](https://github.com/koalaman/shellcheck/wiki/SC2086) : ça n'a pas d'effet en l'état actuel des choses mais on prévient des bugs possible si jamais un nom de fichier contient un jour un caractère de type espace, \n, tabulation, etc. Shellcheck trigger un autre warning sur ce fichier ([SC1117](https://github.com/koalaman/shellcheck/wiki/SC1117)). Je l'ignore délibérément, ce n'est pas vraiment une source de bug et il est sur le point d'être désactivé par shellcheck.

mpepin commented

2018-10-06 16:02:02 +02:00

(Migrated from git.eleves.ens.fr)

added ~64 ~59 ~28 labels

delobell commented

2018-10-06 16:04:22 +02:00

(Migrated from git.eleves.ens.fr)

le printf "OK\n" met la variable $? à 0

le bleu!

> le printf "OK\n" met la variable $? à 0 le bleu!

delobell commented

2018-10-06 16:04:34 +02:00

(Migrated from git.eleves.ens.fr)

merci :)

delobell commented

2018-10-06 16:04:38 +02:00

(Migrated from git.eleves.ens.fr)

merged

delobell commented

2018-10-06 16:04:38 +02:00

(Migrated from git.eleves.ens.fr)

mentioned in commit fc4b852bde

mentioned in commit fc4b852bde3cf30d96b4c428f02901fa3282460d

delobell commented

2018-10-06 17:28:14 +02:00

(Migrated from git.eleves.ens.fr)

Preque @mpepin !

black ... kfet/tests/test_statistic.py" does not exist.

Il prend tous les fichiers comme un seul argument :-( Je peux te laisser retirer les "" dans les calls à black & cie ? (ou une toute solution qui fonctionne que tu trouveras :-P) et merge sur master si t'es sûr ? (je dois aller faire des emplettes)

Preque @mpepin ! > black ... kfet/tests/test_statistic.py" does not exist. Il prend tous les fichiers comme un seul argument :-( Je peux te laisser retirer les "" dans les calls à black & cie ? (ou une toute solution qui fonctionne que tu trouveras :-P) et merge sur master si t'es sûr ? (je dois aller faire des emplettes)